From ac6be9017853140ce52f94da1e54fce926e65edc Mon Sep 17 00:00:00 2001
From: Johannes Millan <johannes.millan@gmail.com>
Date: Wed, 21 Jan 2026 11:01:17 +0100
Subject: [PATCH] refactor(docker): use consistent APP_PORT variable naming

Changed Dockerfile to use APP_PORT instead of PORT to match
docker-entrypoint.sh and nginx template. This prevents confusion
about which environment variable controls the nginx port.
---
 Dockerfile | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index aace96d51..5b6f26652 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -42,7 +42,7 @@ RUN UNSPLASH_KEY=$UNSPLASH_KEY UNSPLASH_CLIENT_ID=$UNSPLASH_CLIENT_ID npm run en
 # Production stage
 FROM nginx:1
 
-ENV PORT=80
+ENV APP_PORT=80
 
 # Install runtime dependencies
 RUN apt-get update && apt-get install -y --no-install-recommends jq && rm -rf /var/lib/apt/lists/*
@@ -52,7 +52,7 @@ COPY --from=build /app/dist/browser /usr/share/nginx/html
 COPY ./nginx/default.conf.template /etc/nginx/templates/default.conf.template
 COPY ./docker-entrypoint.sh /usr/local/bin/docker-entrypoint.sh
 
-EXPOSE $PORT
+EXPOSE $APP_PORT
 WORKDIR /usr/share/nginx/html
 
 ENTRYPOINT ["/usr/local/bin/docker-entrypoint.sh"]