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

docs: addgroup should add system group not user group #3632

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

docs: addgroup should add system group not user group #3632

a1346054 opened this issue Sep 16, 2020 · 4 comments
Assignees
@netblue30
Labels
documentation Issues and pull requests related to the documentation 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
@kmk3 kmk3 added the documentation Issues and pull requests related to the documentation label Aug 24, 2024
@kmk3 kmk3 changed the title addgroup should add system group not user group docs: addgroup should add system group not user group Aug 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@netblue30 netblue30

Labels
documentation Issues and pull requests related to the documentation 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