-
Notifications
You must be signed in to change notification settings - Fork 578
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.
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
deterministic-shutdown option #4635
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Voting against noorphans
because the no prefix suggests that it is an hardening option. Maybe deterministic-shutdown
? Does it terminate/kill child-child processes or does it exit will the child-child processes are still running?
Reminder:
If you add a new command, here's the checklist:
- Update manpages: firejail(1) and firejail-profile(5)
- Update shell completions
- Update vim syntax files
- Update --help
It forcefully shuts down the sandbox if the oldest child in the sandbox dies. Or in other words, it forcefully shuts down the sandbox if the first program, the one that was started by Firejail, dies. The sandbox is shut down even if another process has
Will do, but it will take a while, I'm very slow on these things. |
Now that I'm writing this down it dawns on me that this is not playing nice with |
On the other hand nothing bad will happen. Maybe there is even a use case for |
ea2f03b
to
49766d9
Compare
I guess I like It is less violent than everything I came up with ( |
@smitsohu commented on Oct 26:
A few that had come to mind were
|
Ok, I added a stub to the man pages. I still need to think about if or how to expand it. |
all in! |
Thanks for implementing it. |
- replace `--terminate-orphans` with `--deterministic-shutdown` as it was renamed in the upstream PR: netblue30/firejail#4635
- replace `--terminate-orphans` with `--deterministic-shutdown` as it was renamed in the upstream PR: netblue30/firejail#4635
- replace `--terminate-orphans` with `--deterministic-shutdown` as it was renamed in the upstream PR: netblue30/firejail#4635
* Update firejail path * Fix firejail command options - replace `--terminate-orphans` with `--deterministic-shutdown` as it was renamed in the upstream PR: netblue30/firejail#4635 Co-authored-by: Joel Linn <jl@conductive.de>
Sometimes sandboxed apps are outlived by their child processes which prevents the sandbox from shutting down.
Add a new option to deal with this situation.
Probably there are better names for the option, I just needed a placeholder.
kill-remains
was suggested by @msva.Tracked in #4440
Closes #928
Closes #3042