Merge pull request #5433 from onematchfox/update-nginx-example

Update nginx deployment to use 1.19 features

contrib/docker/frontend-with-nginx/Dockerfile.dockerbuild

@@ -51,9 +51,8 @@ FROM nginx:mainline
 RUN apt-get update && apt-get -y install jq && rm -rf /var/lib/apt/lists/*
 
 COPY --from=build /app/packages/app/dist /usr/share/nginx/html
-COPY docker/default.conf.template /etc/nginx/conf.d/default.conf.template
-COPY docker/run.sh /usr/local/bin/run.sh
+COPY docker/default.conf.template /etc/nginx/templates/default.conf.template
 
-CMD run.sh
+COPY docker/inject-config.sh /docker-entrypoint.d/40-inject-config.sh
 
 ENV PORT 80

contrib/docker/frontend-with-nginx/Dockerfile.hostbuild

@@ -33,9 +33,9 @@ FROM nginx:mainline
 RUN apt-get update && apt-get -y install jq && rm -rf /var/lib/apt/lists/*
 
 COPY packages/app/dist /usr/share/nginx/html
-COPY docker/default.conf.template /etc/nginx/conf.d/default.conf.template
-COPY docker/run.sh /usr/local/bin/run.sh
+COPY docker/default.conf.template /etc/nginx/templates/default.conf.template
 
-CMD run.sh
+COPY docker/inject-config.sh /docker-entrypoint.d/40-inject-config.sh
 
 ENV PORT 80
+

contrib/docker/frontend-with-nginx/docker/inject-config.sh

@@ -2,13 +2,6 @@
 
 set -Eeuo pipefail
 
-# Run nginx as root
-sed -i 's/user  nginx.*$//' /etc/nginx/nginx.conf
-
-# Write selected env vars to nginx config
-envsubst '$PORT' < /etc/nginx/conf.d/default.conf.template > /etc/nginx/conf.d/default.conf
-
-# Inject runtime config into the client
 function inject_config() {
   # Read runtime config from env in the same way as the @backstage/config-loader package
   local config
@@ -41,4 +34,3 @@ function inject_config() {
 
 inject_config
 
-exec nginx -g 'daemon off;'