fix: stringify primitives to avoid regressions

nestjs · Jun 14, 2023 · d2236e6 · d2236e6
1 parent 9f9b3fa
commit d2236e6
Showing 1 changed file with 14 additions and 10 deletions.
diff --git a/lib/config.module.ts b/lib/config.module.ts
@@ -184,8 +184,12 @@ export class ConfigModule {
           config,
         );
         if (options.expandVariables) {
-          const expandOptions: DotenvExpandOptions = typeof options.expandVariables === 'object' ? options.expandVariables : {};
-          config = expand({ ...expandOptions, parsed: config }).parsed || config;
+          const expandOptions: DotenvExpandOptions =
+            typeof options.expandVariables === 'object'
+              ? options.expandVariables
+              : {};
+          config =
+            expand({ ...expandOptions, parsed: config }).parsed || config;
         }
       }
     }
@@ -197,14 +201,14 @@ export class ConfigModule {
       return;
     }
     const keys = Object.keys(config).filter(key => !(key in process.env));
-    keys.forEach(
-      key => {
-        const value = config[key];
-        if (typeof value === 'string') {
-          process.env[key] = value;
-        }
-      },
-    );
+    keys.forEach(key => {
+      const value = config[key];
+      if (typeof value === 'string') {
+        process.env[key] = value;
+      } else if (typeof value === 'boolean' || typeof value === 'number') {
+        process.env[key] = `${value}`;
+      }
+    });
   }
 
   private static mergePartial(