feat: Remove unused build arguments and simplify Nginx configuration for improved performance

Author: Aloento

.github/workflows/docker-build-push-on-tag-or-release.yaml

@@ -50,6 +50,3 @@ jobs:
           tags: ${{ steps.meta.outputs.tags }}
           labels: ${{ steps.meta.outputs.labels }}
           push: true
-          build-args: |
-            BASE_URL=${{ secrets.BASE_URL }}
-            AUTH_TOKEN=${{ secrets.AUTH_TOKEN }}

Dockerfile

@@ -14,8 +14,6 @@ RUN pnpm run build
 
 FROM nginx:stable-alpine
 
-RUN sed -i '1idaemon off;' /etc/nginx/nginx.conf
-
 COPY nginx.conf /etc/nginx/conf.d/default.conf
 
 COPY --from=prod /app/dist/ /app

nginx.conf

@@ -1,25 +1,19 @@
 server {
 	listen 80 default_server;
 
-	gzip on;
-	gzip_min_length 1000;
-	gzip_types text/plain text/xml application/javascript text/css;
-
 	root /app;
 
 	# normal routes
 	# serve given url and default to index.html if not found
 	# e.g. /, /user and /foo/bar will return index.html
 	location / {
-		add_header Cache-Control "no-store";
 		try_files $uri $uri/index.html /index.html;
 	}
 
 	# files
 	# for all routes matching a dot, check for files and return 404 if not found
 	# e.g. /file.js returns a 404 if not found
 	location ~ \.(?!html) {
-		add_header Cache-Control "public, max-age=2678400";
 		try_files $uri =404;
 	}
 }