Skip to content

PROD - Check Informix for reviews, if we're missing that data #351

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 18 commits into from
May 17, 2024
Merged

PROD - Check Informix for reviews, if we're missing that data #351

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
795423a
Add Informix fallback when retrieving reviews (PS-268)
jmgasper May 15, 2024
3b93967
Lower version to support the Informix NPM package. It's not compatib…
jmgasper May 15, 2024
8f04dec
Merge branch 'develop' of github.com:topcoder-platform/submissions-ap…
jmgasper May 15, 2024
1eb4e23
Remove hard-coded testing data for review types
jmgasper May 15, 2024
131504c
Remove debug code
jmgasper May 15, 2024
f686b9e
Fix odd typo
jmgasper May 15, 2024
fe01066
Minor cleanup and documentation
jmgasper May 15, 2024
47a3667
Fix node version in Docker
jmgasper May 15, 2024
0baf479
Fix node engine
jmgasper May 15, 2024
a6275c8
Use npm instead of yarn due to compatibility issues
jmgasper May 15, 2024
7068add
Lets try this
jmgasper May 16, 2024
2a771cc
Small tweak
jmgasper May 16, 2024
fca7d01
Tweaks for feedback to PR
jmgasper May 16, 2024
b6d1f33
Trying this
jmgasper May 16, 2024
ec6678e
Better use of ENV variables for Informix libs
jmgasper May 16, 2024
4f4e048
Additional ID check
jmgasper May 16, 2024
a902478
Fix legacy ID check
jmgasper May 16, 2024
24c3ab3
Fix a problem with review summations during ongoing challenge
jmgasper May 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 16 additions & 1 deletion config/default.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,5 +55,20 @@ module.exports = {
'd6d31f34-8ee5-4589-ae65-45652fcc01a6': 30000720
},

INTERNAL_CACHE_TTL: process.env.INTERNAL_CACHE_TTL || 1800
INTERNAL_CACHE_TTL: process.env.INTERNAL_CACHE_TTL || 1800,
INFORMIX: {
SERVER: process.env.INFORMIX_SERVER || 'informixoltp_tcp', // informix server
DATABASE: process.env.INFORMIX_DATABASE || 'tcs_catalog', // informix database
HOST: process.env.INFORMIX_HOST || 'localhost', // host
PROTOCOL: process.env.INFORMIX_PROTOCOL || 'onsoctcp',
PORT: process.env.INFORMIX_PORT || '2021', // port
DB_LOCALE: process.env.INFORMIX_DB_LOCALE || 'en_US.57372',
USER: process.env.INFORMIX_USER || 'informix', // user
PASSWORD: process.env.INFORMIX_PASSWORD || '1nf0rm1x', // password
POOL_MAX_SIZE: parseInt(process.env.MAXPOOL, 10) || 60,
maxsize: parseInt(process.env.MAXSIZE) || 0,
minpool: parseInt(process.env.MINPOOL, 10) || 1,
idleTimeout: parseInt(process.env.IDLETIMEOUT, 10) || 3600,
timeout: parseInt(process.env.TIMEOUT, 10) || 30000
}
}
6 changes: 4 additions & 2 deletions docker/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# Use the base image with Node.js
FROM node:16.20.1-buster
FROM node:14.21.3

# Copy the current directory into the Docker image
COPY . /submissions-api
Expand All @@ -8,5 +8,7 @@ COPY . /submissions-api
WORKDIR /submissions-api

# Install the dependencies from package.json
RUN yarn install
RUN npm install
ENV INFORMIXDIR=/submissions-api/node_modules/informixdb/installer/onedb-odbc-driver
ENV LD_LIBRARY_PATH=${INFORMIXDIR}/lib:${INFORMIXDIR}/lib/esql:${INFORMIXDIR}/lib/cli
ENTRYPOINT node app.js
Loading