nginx-proxy-manager icon indicating copy to clipboard operation
nginx-proxy-manager copied to clipboard

Enhance PostgreSQL configuration support in the backend

Open anvarknian opened this issue 2 months ago • 3 comments

  • 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 avatar Sep 27 '25 22:09 anvarknian

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/[email protected]" 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/[email protected]" 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

anvarknian avatar Sep 27 '25 23:09 anvarknian

The react branch was merged to develop which included removing eslint and improvements to the backend codebase. Please rebase, or if it's easier, create a new PR.

Also curious if you've tested with a non-public schema, and what happens if the schema doesn't exist prior?

jc21 avatar Nov 02 '25 22:11 jc21