Skip to content

Commit

Permalink
Run node server from app user
Browse files Browse the repository at this point in the history
  • Loading branch information
Carlgo11 committed Jan 14, 2025
1 parent 93434bc commit a14e624
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
4 changes: 3 additions & 1 deletion inbox/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,12 @@ ENV INBOX_TLS_CERT_PATH="/certs/inbox/cert.pem"
ENV REDIS_HOST="redis_mail"
ENV REDIS_PORT="6379"

RUN apk add openssl
RUN apk add openssl; adduser app -H -D

WORKDIR /usr/src/inbox

COPY --from=build /app /usr/src/inbox

USER app

ENTRYPOINT "node src/index.js 2>&1 | tee /tmp/inbox.log"
4 changes: 3 additions & 1 deletion inbox/Dockerfile.dev
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,13 @@ ENV INBOX_TLS_CERT_PATH="/certs/inbox/cert.pem"
ENV REDIS_HOST="redis_mail"
ENV REDIS_PORT="6379"

RUN apk add openssl
RUN apk add openssl; adduser app -H -D

WORKDIR /usr/src/inbox

COPY --from=build /app /usr/src/inbox
COPY ./smtp-server /usr/src/inbox/node_modules/@carlgo11/smtp-server

USER app

ENTRYPOINT "node src/index.js 2>&1 | tee /tmp/inbox.log"

Check warning on line 27 in inbox/Dockerfile.dev

View workflow job for this annotation

GitHub Actions / docker

JSON arguments recommended for ENTRYPOINT/CMD to prevent unintended behavior related to OS signals

JSONArgsRecommended: JSON arguments recommended for ENTRYPOINT to prevent unintended behavior related to OS signals More info: https://docs.docker.com/go/dockerfile/rule/json-args-recommended/
2 changes: 1 addition & 1 deletion inbox/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
"node-forge": "^1.3.1",
"nodemailer": "^6.9.14",
"redis": "^4.7.0",
"@carlgo11/smtp-server": "^0.0.7"
"@carlgo11/smtp-server": "^0.0.9"
},
"engines": {
"node": ">=16"
Expand Down

0 comments on commit a14e624

Please sign in to comment.