Переглянути джерело

fix(config): always use internal port 5432 for postgres in non-test mode

Nicolas Meienberger 1 рік тому
батько
коміт
ef541f1d77

+ 1 - 1
src/app/(dashboard)/app-store/[id]/components/InstallForm/InstallForm.tsx

@@ -175,7 +175,7 @@ export const InstallForm: React.FC<IProps> = ({ formFields, info, onSubmit, init
         {initalValues ? t('submit-update') : t('sumbit-install')}
         {initalValues ? t('submit-update') : t('sumbit-install')}
       </Button>
       </Button>
       {initalValues && onReset && (
       {initalValues && onReset && (
-        <Button loading={status === 'resetting'} onClick={onClickReset} className="btn-danger ms-2">
+        <Button loading={status === 'stopping'} onClick={onClickReset} className="btn-danger ms-2">
           {t('reset')}
           {t('reset')}
         </Button>
         </Button>
       )}
       )}

+ 10 - 3
src/server/core/TipiConfig/TipiConfig.ts

@@ -29,12 +29,12 @@ export class TipiConfig {
     const envMap = envStringToMap(envFile.toString());
     const envMap = envStringToMap(envFile.toString());
 
 
     const conf = { ...process.env, ...Object.fromEntries(envMap), ...nextConfig().serverRuntimeConfig };
     const conf = { ...process.env, ...Object.fromEntries(envMap), ...nextConfig().serverRuntimeConfig };
-    const envConfig: z.infer<typeof envSchema> = {
+    const envConfig: z.input<typeof envSchema> = {
       postgresHost: conf.POSTGRES_HOST,
       postgresHost: conf.POSTGRES_HOST,
       postgresDatabase: conf.POSTGRES_DBNAME,
       postgresDatabase: conf.POSTGRES_DBNAME,
       postgresUsername: conf.POSTGRES_USERNAME,
       postgresUsername: conf.POSTGRES_USERNAME,
       postgresPassword: conf.POSTGRES_PASSWORD,
       postgresPassword: conf.POSTGRES_PASSWORD,
-      postgresPort: Number(conf.POSTGRES_PORT || 5432),
+      postgresPort: Number(conf.POSTGRES_PORT),
       REDIS_HOST: conf.REDIS_HOST,
       REDIS_HOST: conf.REDIS_HOST,
       redisPassword: conf.REDIS_PASSWORD,
       redisPassword: conf.REDIS_PASSWORD,
       NODE_ENV: conf.NODE_ENV,
       NODE_ENV: conf.NODE_ENV,
@@ -84,7 +84,14 @@ export class TipiConfig {
   }
   }
 
 
   public getConfig() {
   public getConfig() {
-    return { ...this.config, ...this.getFileConfig() };
+    let conf = { ...this.config, ...this.getFileConfig() };
+
+    // If we are not in test mode, we need to set the postgres port to 5432 (internal port)
+    if (conf.NODE_ENV !== 'test') {
+      conf = { ...conf, postgresPort: 5432 };
+    }
+
+    return conf;
   }
   }
 
 
   public getSettings() {
   public getSettings() {

+ 1 - 1
src/server/run-migrations-dev.ts

@@ -17,7 +17,7 @@ export const runPostgresMigrations = async (dbName?: string) => {
     host: postgresHost,
     host: postgresHost,
     database: dbName || postgresDatabase,
     database: dbName || postgresDatabase,
     password: postgresPassword,
     password: postgresPassword,
-    port: Number(postgresPort),
+    port: Number(process.env.POSTGRES_PORT),
   });
   });
   await client.connect();
   await client.connect();