Skip to content

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.

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

Jump to bottom
Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Enhance PostgreSQL configuration support in the backend #4779

wants to merge 5 commits into from
+68 −28

Conversation

anvarknian
Copy link

  • 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.

@anvarknian
Enhance PostgreSQL configuration support in the backend
0c0c165 
- 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.
@anvarknian
Refactor PostgreSQL configuration handling in backend
d8d7469 
- 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.
@anvarknian
Refactor code style in backend configuration files
c887dd4 
- 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.
@nginxproxymanagerci
Copy link

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.

@anvarknian
Copy link
Author

anvarknian commented Sep 27, 2025 •
edited
Loading

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
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@anvarknian