Skip to content

🐛 Bug Report: Node user instead of root user in Dockerfiles #4480

@mictrax

Description

@mictrax

📜 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?

Are you willing to submit PR?

Yes I am willing to submit a PR!

Metadata

Metadata

Assignees

Labels

help wantedExtra attention is needed

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions