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

addgroup should add system group not user group #3632

Open
a1346054 opened this issue Sep 16, 2020 · 4 comments
Open

addgroup should add system group not user group #3632

a1346054 opened this issue Sep 16, 2020 · 4 comments
Assignees
@netblue30
Labels
wordpress This is related to the official website

Comments

@a1346054
Copy link
Contributor

On https://firejail.wordpress.com/documentation-2/basic-usage/

it states:

# addgroup firejail
# chown root:firejail /usr/bin/firejail
# chmod 4750 /usr/bin/firejail

Would it not be preferable to use addgroup --system to create a system group with a GID < 1000 instead of a regular user group with a GID > 1000?
@rusty-snake
Copy link
Collaborator

@netblue30 ??

netblue30 pushed a commit that referenced this issue Oct 3, 2020
move to addgroup --system (#3632)
ce7dd15
@netblue30 netblue30 added the in testing A bugfix that is being tested label Oct 3, 2020
@netblue30
Copy link
Owner

I'll leave it open for now to remember to modify the web page.

@a1346054
Copy link
Contributor Author

Any progress on this?

@matu3ba matu3ba mentioned this issue Oct 7, 2021
Closed
@rusty-snake rusty-snake removed the in testing A bugfix that is being tested label Jun 21, 2022
@kmk3 kmk3 added the wordpress This is related to the official website label Jun 22, 2022
@a1346054
Copy link
Contributor Author

I'll leave it open for now to remember to modify the web page.

https://firejail.wordpress.com/documentation-2/basic-usage/ still uses addgroup without --system hence creating groups with gid >1000 (on debian), interfering with usual expectations.

@rusty-snake rusty-snake reopened this Sep 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@netblue30 netblue30

Labels
wordpress This is related to the official website
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@netblue30 @kmk3 @a1346054 @rusty-snake