-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Open
Labels
help wantedExtra attention is neededExtra attention is needed
Description
📜 Description
For security reasons, it would be great to switch to the node user (1000) and grant the permissions instead of using the root user in the Dockerfiles (especially in the Dockerfile for the web container)
Set the permissions for the current folder to the node user and switch user at the end of the dockerfiles
RUN chown -R 1000:1000 .
USER 1000
👟 Reproduction steps
Build docker images
👍 Expected behavior
Node user (1000) should be used instead of root user
👎 Actual Behavior with Screenshots
Folder permissions are not set for node user
Novu version
0.19.0
npm version
No response
node version
No response
📃 Provide any additional context for the Bug.
No response
👀 Have you spent some time to check if this bug has been raised before?
- I checked and didn't find a similar issue
🏢 Have you read the Contributing Guidelines?
- I have read the Contributing Guidelines
Are you willing to submit PR?
Yes I am willing to submit a PR!
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
help wantedExtra attention is neededExtra attention is needed