Merge pull request #298 from TaloDev/docker-substitution-step

Prevent env vars being substituted with empty at Dockerfile level
TaloDev · Oct 8, 2024 · c7ee522 · c7ee522
2 parents d317c6c + e8c57fd
commit c7ee522
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/.env.production b/.env.production
@@ -1,4 +1,4 @@
-VITE_API_URL=\${API_URL}
+VITE_API_URL=${API_URL}
 
-VITE_SENTRY_DSN=\${SENTRY_DSN}
+VITE_SENTRY_DSN=${SENTRY_DSN}
 VITE_SENTRY_ENV=prod
diff --git a/Dockerfile b/Dockerfile
@@ -1,6 +1,11 @@
 FROM node:20-alpine AS build
 WORKDIR /usr/frontend
 COPY . .
+
+# prepend \ to variables that need to be substituted
+# this prevents them from being substituted with empty values
+RUN sed -i 's/\${/\\${/g' .env.production
+
 RUN npm install
 RUN npm run build