kevin/portfolio
1
0
Fork 0
Code

Do not expose port 443 in internal dockerfile.

Browse Source 
It is no longer needed, since all HTTPS traffic will be handled by the Nginx service on the server hosting the docker containers.
This commit is contained in:
Kevin Matsubara 2024-12-31 10:32:09 +01:00
parent 97eedf1345
commit 266653b48b
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Dockerfile
View File

@ -1,5 +1,5 @@
# stage 1 - build react and elm apps first # 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 &&\ RUN apk update &&\
apk --no-cache add curl &&\ apk --no-cache add curl &&\
@ -10,7 +10,7 @@ RUN apk update &&\
apk del curl apk del curl
WORKDIR /app WORKDIR /app
ENV PATH /app/node_modules/.bin:$PATH ENV PATH=/app/node_modules/.bin:$PATH
COPY ./package.json /app/ COPY ./package.json /app/
RUN npm install RUN npm install
COPY . /app 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 RUN rm /etc/nginx/conf.d/default.conf
COPY config/nginx/nginx.conf /etc/nginx/conf.d COPY config/nginx/nginx.conf /etc/nginx/conf.d
EXPOSE 80 EXPOSE 80
EXPOSE 443 #EXPOSE 443
CMD ["nginx", "-g", "daemon off;"] CMD ["nginx", "-g", "daemon off;"]