Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change to a non-root user. #6

Closed
wants to merge 1 commit into from

Conversation

RJSzynal
Copy link

If you already have a volume created, this will need to be chown'd to 1000:1000 or the uid/gid overriden at run time

If you already have a volume created, this will need to be chown'd to 1000:1000 or the uid/gid overriden at run time
@beberg
Copy link
Contributor

beberg commented Apr 25, 2020

1000:1000 isnt always the right choice. Clarified in READMEs, see issue for explanation.

@beberg beberg closed this Apr 25, 2020
@RJSzynal
Copy link
Author

I would argue that while 1000:1000 isn't always the right choice (nothing is), root is always the wrong choice.
I don't think you can rely on all users to properly read the readme so it is important to ensure the very base defaults are the most secure, especially when it's something that many people around the world will be using to provide charitable help to the cause and perhaps don't have much experience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants