From 1a3ee304c7cafddd6cfe0ca8e5112929570c3ba8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Przemys=C5=82aw=20=C5=BBydek?= <przemyslawzydek@gmail.com>
Date: Wed, 22 Nov 2023 12:04:09 +0200
Subject: [PATCH] feat: validate env values on build

---
 rollup.config.mjs | 22 +++++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

diff --git a/rollup.config.mjs b/rollup.config.mjs
index 13b9348a..3ab3fa01 100644
--- a/rollup.config.mjs
+++ b/rollup.config.mjs
@@ -13,6 +13,22 @@ import packageJson from './package.json' assert { type: 'json' }
 dotenv.config()
 const outputDirectory = 'dist'
 
+function getEnv(key, defaultValue) {
+  const value = process.env[key]
+
+  if (!value && !defaultValue) {
+    throw new Error(`Missing environment variable ${key}`)
+  }
+
+  if (!value) {
+    console.warn(`Missing environment variable "${key}". Using default value: ${defaultValue}`)
+
+    return defaultValue
+  }
+
+  return value
+}
+
 function makeConfig(opts, entryFile, artifactName, functionJsonPath, transformFunctionJson) {
   const isDev = opts.watch
 
@@ -33,8 +49,8 @@ function makeConfig(opts, entryFile, artifactName, functionJsonPath, transformFu
   })
 
   const env = {
-    fpcnd: process.env.FPCDN ?? 'fpcdn.io',
-    ingressApi: process.env.INGRESS_API ?? 'api.fpjs.io',
+    fpcdn: getEnv('FPCDN', 'fpcdn.io'),
+    ingressApi: getEnv('INGRESS_API', 'api.fpjs.io'),
   }
 
   /**
@@ -74,7 +90,7 @@ function makeConfig(opts, entryFile, artifactName, functionJsonPath, transformFu
       commonjs(),
       nodeResolve({ preferBuiltins: false, exportConditions: ['node'] }),
       replace({
-        __FPCDN__: env.fpcnd,
+        __FPCDN__: env.fpcdn,
         __INGRESS_API__: env.ingressApi,
         __azure_function_version__: packageJson.version,
         preventAssignment: true,