fix(server): support non-default postgres port when taking a backup (#…

…13992)
immich-app · Nov 7, 2024 · 7534098 · 7534098
1 parent ec5b7c2
commit 7534098
Showing 1 changed file with 21 additions and 3 deletions.
diff --git a/server/src/services/backup.service.ts b/server/src/services/backup.service.ts
@@ -80,16 +80,34 @@ export class BackupService extends BaseService {
     } = this.configRepository.getEnv();
 
     const isUrlConnection = config.connectionType === 'url';
-    const databaseParams = isUrlConnection ? ['-d', config.url] : ['-U', config.username, '-h', config.host];
+
+    const databaseParams = isUrlConnection
+      ? ['--dbname', config.url]
+      : [
+          '--username',
+          config.username,
+          '--host',
+          config.host,
+          '--port',
+          `${config.port}`,
+          '--database',
+          config.database,
+        ];
+
+    databaseParams.push('--clean', '--if-exists');
+
     const backupFilePath = path.join(
       StorageCore.getBaseFolder(StorageFolder.BACKUPS),
       `immich-db-backup-${Date.now()}.sql.gz.tmp`,
     );
 
     try {
       await new Promise<void>((resolve, reject) => {
-        const pgdump = this.processRepository.spawn(`pg_dumpall`, [...databaseParams, '--clean', '--if-exists'], {
-          env: { PATH: process.env.PATH, PGPASSWORD: isUrlConnection ? undefined : config.password },
+        const pgdump = this.processRepository.spawn(`pg_dumpall`, databaseParams, {
+          env: {
+            PATH: process.env.PATH,
+            PGPASSWORD: isUrlConnection ? undefined : config.password,
+          },
         });
 
         // NOTE: `--rsyncable` is only supported in GNU gzip