Skip to content

Adds instructions for modifying the node user#328

Merged
SimenB merged 1 commit intonodejs:masterfrom
cody-vibe:docs/modify-user
Jul 25, 2017
Merged

Adds instructions for modifying the node user#328
SimenB merged 1 commit intonodejs:masterfrom
cody-vibe:docs/modify-user

Conversation

@cody-vibe
Copy link
Copy Markdown
Contributor

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.

@cody-vibe
Copy link
Copy Markdown
Contributor Author

@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
Copy Markdown
Member

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

@cody-vibe
Copy link
Copy Markdown
Contributor Author

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

@SimenB
Copy link
Copy Markdown
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
@cody-vibe
Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
Member

SimenB commented Jul 25, 2017

Thanks!

@SimenB
Copy link
Copy Markdown
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

@cody-vibe
Copy link
Copy Markdown
Contributor Author

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