-
Couldn't load subscription status.
- Fork 3.3k
Enhance PostgreSQL configuration support in the backend #4779
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.
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
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
@anvarknian
Copy link
@anvarknian
anvarknian
commented
Sep 27, 2025
- Updated db.js to include PostgreSQL-specific connection options such as schema and SSL mode.
- Modified config.js to read new environment variables for PostgreSQL schema and SSL mode.
- Added PostgreSQL configuration section in knexfile.js.
- Updated docker-compose.dev.yml and documentation to reflect new environment variables for PostgreSQL schema and SSL mode.
- Updated db.js to include PostgreSQL-specific connection options such as schema and SSL mode. - Modified config.js to read new environment variables for PostgreSQL schema and SSL mode. - Added PostgreSQL configuration section in knexfile.js. - Updated docker-compose.dev.yml and documentation to reflect new environment variables for PostgreSQL schema and SSL mode.
- Cleaned up formatting in db.js for better readability of SSL mode handling. - Aligned variable declarations in config.js for consistency and clarity. - No functional changes were made, only code style improvements.
- Improved formatting in db.js and config.js for better readability and consistency. - Aligned variable declarations and indentation throughout both files. - No functional changes were made, focusing solely on code style enhancements.
CI Error:
/bin/bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)
certbot-node: Pulling from nginxproxymanager/nginx-full
Digest: sha256:41f3f7a330e3b4de82d8f20988ed15adc4828926d522859fb8fe6189c99868bc
Status: Image is up to date for nginxproxymanager/nginx-full:certbot-node
docker.io/nginxproxymanager/nginx-full:certbot-node
�[1;34m❯ �[1;36mTesting backend ...�[0m
yarn install v1.22.22
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
warning " > @apidevtools/swagger-parser@10.1.0" has unmet peer dependency "openapi-types@>=7".
[4/4] Building fresh packages...
Done in 37.56s.
yarn run v1.22.22
$ /app/node_modules/.bin/eslint .
/app/db.js
13:9 error Missing space before value for key 'host' key-spacing
14:9 error Missing space before value for key 'user' key-spacing
17:9 error Missing space before value for key 'port' key-spacing
34:11 error Missing space before value for key 'client' key-spacing
/app/lib/config.js
1:1 error This group of assignments is not aligned align-assignments/align-assignments
5:1 error This group of assignments is not aligned align-assignments/align-assignments
26:4 error This group of assignments is not aligned align-assignments/align-assignments
40:13 error Missing space before value for key 'engine' key-spacing
41:11 error Missing space before value for key 'host' key-spacing
42:11 error Missing space before value for key 'port' key-spacing
43:11 error Missing space before value for key 'user' key-spacing
45:11 error Missing space before value for key 'name' key-spacing
55:2 error This group of assignments is not aligned align-assignments/align-assignments
58:2 error This group of assignments is not aligned align-assignments/align-assignments
68:13 error Missing space before value for key 'engine' key-spacing
69:11 error Missing space before value for key 'host' key-spacing
70:11 error Missing space before value for key 'port' key-spacing
71:11 error Missing space before value for key 'user' key-spacing
73:11 error Missing space before value for key 'name' key-spacing
74:13 error Missing space before value for key 'schema' key-spacing
75:14 error Missing space before value for key 'sslMode' key-spacing
87:10 error Missing space before value for key 'knex' key-spacing
88:13 error Missing space before value for key 'client' key-spacing
144:3 error This group of assignments is not aligned align-assignments/align-assignments
✖ 24 problems (24 errors, 0 warnings)
24 errors and 0 warnings potentially fixable with the `--fix` option.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
On my side everything is working fine, including Eslint.
Probably CI must be fixed
yarn install v1.22.22
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
warning " > @apidevtools/swagger-parser@10.1.0" has unmet peer dependency "openapi-types@>=7".
[4/4] Building fresh packages...
Done in 44.88s.
yarn run v1.22.22
$ /app/node_modules/.bin/eslint .
Done in 1.11s.
\E[1;34m❯ \E[1;32mTesting Complete\E[0m
\E[1;34m❯ \E[1;36mBuilding ...\E[0m
#0 building with "desktop-linux" instance using docker driver
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.