make sure we dont accept non-empty string

rdmurphy · Apr 24, 2023 · b4dc8ab · b4dc8ab
1 parent 5137b42
commit b4dc8ab
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/packages/pds/src/config.ts b/packages/pds/src/config.ts
@@ -144,7 +144,9 @@ export class ServerConfig {
     )
 
     // E.g. ws://abc.com:4000
-    const appViewRepoProvider = process.env.APP_VIEW_REPO_PROVIDER || undefined
+    const appViewRepoProvider = nonemptyString(
+      process.env.APP_VIEW_REPO_PROVIDER,
+    )
 
     return new ServerConfig({
       debugMode,
@@ -357,6 +359,11 @@ export class ServerConfig {
   }
 }
 
+const nonemptyString = (str: string | undefined): string | undefined => {
+  if (str === undefined || str.length === 0) return undefined
+  return str
+}
+
 const parseIntWithFallback = <T>(
   value: string | undefined,
   fallback: T,