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.

Sign up for GitHub

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

Jump to bottom

fix: Set uid/gid in docker image to 0 #8908

Merged
merged 2 commits into from Feb 20, 2023
Merged

fix: Set uid/gid in docker image to 0 #8908

merged 2 commits into from Feb 20, 2023

Conversation

rroesch1
Copy link
Contributor

  • UIDs/GIDs in the docker image are messed up due to strange UIDs/GIDs in tarballs from the npm registry
  • This change overrides the UID/GID of all files in /usr/src/app

fixes: #8906

@rroesch1
fix: Set uid/gid in docker image to 0
f279c7d 
* UIDs/GIDs in the docker image are messed up due to strange UIDs/GIDs in tarballs from the npm registry
* This change overrides the UID/GID of all files in `/usr/src/app`

fixes: badges#8906
@github-actions
Copy link
Contributor

github-actions bot commented Feb 15, 2023 •
edited

Messages
📖 ✨ Thanks for your contribution to Shields, @rroesch1!

Generated by 🚫 dangerJS against 7ec1a23

@rroesch1 rroesch1 mentioned this pull request Feb 15, 2023
Closed
@chris48s
Copy link
Member

Cheers. I tested this locally and it doesn't appear to introduce any problems

@chris48s chris48s added the self-hosting Discussion, problems, features, and documentation related to self-hosting Shields label Feb 20, 2023
@chris48s chris48s merged commit 174b191 into badges:master Feb 20, 2023
@sunchengzhu sunchengzhu mentioned this pull request Apr 10, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chris48s chris48s chris48s approved these changes

Assignees
No one assigned
Labels
self-hosting Discussion, problems, features, and documentation related to self-hosting Shields
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

File UIDs/GIDs in docker image messed up since server-2022-12-01
2 participants
@rroesch1 @chris48s