From 266653b48b8604001131d9a474956ce510918217 Mon Sep 17 00:00:00 2001 From: Kevin Matsubara Date: Tue, 31 Dec 2024 10:32:09 +0100 Subject: [PATCH] Do not expose port 443 in internal dockerfile. It is no longer needed, since all HTTPS traffic will be handled by the Nginx service on the server hosting the docker containers. --- Dockerfile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Dockerfile b/Dockerfile index 5221bc7..a86d6f5 100644 --- a/Dockerfile +++ b/Dockerfile @@ -1,5 +1,5 @@ # stage 1 - build react and elm apps first -FROM node:20.5.1-alpine as build +FROM node:20.5.1-alpine AS build RUN apk update &&\ apk --no-cache add curl &&\ @@ -10,7 +10,7 @@ RUN apk update &&\ apk del curl WORKDIR /app -ENV PATH /app/node_modules/.bin:$PATH +ENV PATH=/app/node_modules/.bin:$PATH COPY ./package.json /app/ RUN npm install COPY . /app @@ -25,5 +25,5 @@ COPY --from=build /app/src/michelaben /usr/share/nginx/html/michelaben RUN rm /etc/nginx/conf.d/default.conf COPY config/nginx/nginx.conf /etc/nginx/conf.d EXPOSE 80 -EXPOSE 443 +#EXPOSE 443 CMD ["nginx", "-g", "daemon off;"]