Skip to content

Commit

Permalink
Merge pull request #1560 from argos-ci/add-pg-port-config
Browse files Browse the repository at this point in the history
chore: fix warning by adding pg port to config
  • Loading branch information
gregberge authored Feb 9, 2025
2 parents e59d912 + a3fdb88 commit eb8263d
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion apps/backend/src/config/database.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ export function loadDatabaseConfigFromURL(url: string, config: Config) {
const urlObj = new URL(url);

config.set("pg.connection.host", urlObj.hostname);
config.set("pg.connection.port", urlObj.port);
config.set("pg.connection.port", urlObj.port ? Number(urlObj.port) : 5432);
config.set("pg.connection.user", urlObj.username);
config.set("pg.connection.password", urlObj.password);
config.set("pg.connection.database", urlObj.pathname.substring(1));
Expand Down Expand Up @@ -39,6 +39,7 @@ export function getKnexConfig(config: Config): Knex.Config {
database: config.get("pg.connection.database"),
host: config.get("pg.connection.host"),
user: config.get("pg.connection.user"),
port: config.get("pg.connection.port"),
password: config.get("pg.connection.password") || "",
timezone: "utc",
ssl: config.get("pg.connection.ssl")
Expand Down

0 comments on commit eb8263d

Please sign in to comment.