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

Default to high user ID #3277

Closed
rarkins opened this issue Sep 19, 2024 · 3 comments · Fixed by #3351
Closed

Default to high user ID #3277

rarkins opened this issue Sep 19, 2024 · 3 comments · Fixed by #3351
Assignees
@viceice
Labels
breaking Breaking change, requires major version bump priority-2-important User-visible bugs or very important features status:in-progress Someone is working on implementation type:feature Feature (new functionality)
Milestone
v12

Comments

@rarkins
Copy link
Member

rarkins commented Sep 19, 2024

Some security guides/scanners recommend running containers with a high user ID.

Refs:

Particularly, user 1000 is one of the most likely to have a conflict, but I think we can pick a really high one semi-randomly so that the chance a user needs to use it for something else already is close to zero.
@rarkins rarkins added the breaking Breaking change, requires major version bump label Sep 19, 2024
@viceice
Copy link
Member

viceice commented Sep 19, 2024

do you have a preference?

@rarkins
Copy link
Member Author

rarkins commented Sep 19, 2024

Pick a random high number?

@viceice
Copy link
Member

viceice commented Sep 30, 2024

i'll pick 12021 because it's higer than 10000 and this repo was created at 2021 🙃

@viceice viceice added this to the v12 milestone Sep 30, 2024
viceice added a commit that referenced this issue Sep 30, 2024
@viceice
feat!: switch default userid to 12021
230ddb3 
- closes #3277
@viceice viceice mentioned this issue Sep 30, 2024
Merged
viceice added a commit that referenced this issue Sep 30, 2024
@viceice
feat!: switch default userid to 12021
1136133 
- closes #3277
@viceice viceice mentioned this issue Oct 1, 2024
Merged
@viceice viceice added status:in-progress Someone is working on implementation type:feature Feature (new functionality) priority-2-important User-visible bugs or very important features labels Oct 1, 2024
viceice added a commit that referenced this issue Oct 1, 2024
@viceice
feat!: switch default userid to 12021
447d6c0 
- closes #3277
viceice added a commit that referenced this issue Oct 1, 2024
@viceice
feat!: switch default userid to 12021 (#3351)
c31ba08 
- closes #3277

BREAKING CHANGE: the default user id is now `12021`.
@viceice viceice closed this as completed Oct 1, 2024
viceice added a commit that referenced this issue Oct 1, 2024
@viceice
feat!: switch default userid to 12021 (#3351)
e8a6080 
- closes #3277

BREAKING CHANGE: the default user id is now `12021`.
viceice added a commit that referenced this issue Oct 1, 2024
@viceice
feat!: switch default userid to 12021 (#3351)
563c0d4 
- closes #3277

BREAKING CHANGE: the default user id is now `12021`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@viceice viceice

Labels
breaking Breaking change, requires major version bump priority-2-important User-visible bugs or very important features status:in-progress Someone is working on implementation type:feature Feature (new functionality)
Projects
None yet
Milestone
v12
Development

Successfully merging a pull request may close this issue.

feat!: switch default userid to 12021 containerbase/base
2 participants
@viceice @rarkins