Skip to content

Adds instructions for modifying the node user #328

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

Merged
merged 1 commit into from
Jul 25, 2017
Merged

Adds instructions for modifying the node user #328

merged 1 commit into from
Jul 25, 2017

Conversation

sdwolfz
Copy link
Contributor

@sdwolfz sdwolfz commented Feb 9, 2017

Also highlights the Best Practices section in the README.md file so
the new instructions are easier to find.

Special thanks to:

Refs #289

Starefossen

This comment was marked as off-topic.

Starefossen

This comment was marked as off-topic.

@sdwolfz
Copy link
Contributor Author

sdwolfz commented Feb 9, 2017

@LaurentGoderre I was unable to find a way to modify a group uid in busybox, do you have any idea how to do it, or should I simply remove it and create another like below?

# For debian based images use:
RUN groupmod -g 999 node && usermod -u 999 -g 999 node

# For alpine based images use:
RUN deluser --remove-home node \
  && delgroup node \
  && addgroup -S node -g 999 \
  && adduser -S -g node -u 999 node 

@LaurentGoderre
Copy link
Member

@SDWolf sorry for the delay, which one were you having issues with, the alpine one?

@sdwolfz
Copy link
Contributor Author

sdwolfz commented Mar 30, 2017

Yes, for alpine there does not seem to be a way, AFAIK, to update a user's name/uid.

@SimenB
Copy link
Member

SimenB commented Jul 25, 2017

@SDWolf separate instructions for alpine seems to be the way to go 😄

styfle

This comment was marked as off-topic.

SimenB

This comment was marked as off-topic.

Also highlights the `Best Practices` section in the `README.md` file so
the new instructions are easier to find.

Special thanks to:
* @blueimp for the uid/gid change command
* @LaurentGoderre for the user rename command

Refs #289
@sdwolfz
Copy link
Contributor Author

sdwolfz commented Jul 25, 2017

I've updated the PR and squashed the commits, let me know if there is anything else to be done 😃.

@SimenB SimenB merged commit c3694d8 into nodejs:master Jul 25, 2017
@SimenB
Copy link
Member

SimenB commented Jul 25, 2017

Thanks!

@SimenB
Copy link
Member

SimenB commented Jul 25, 2017

@SDWolf could you send a PR fixing the ToC in the readme?

See https://travis-ci.org/nodejs/docker-node/jobs/257311345

@sdwolfz
Copy link
Contributor Author

sdwolfz commented Jul 25, 2017

Opened #475.

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.

5 participants