mirror of
https://github.com/netblue30/firejail.git
synced 2026-05-15 14:16:14 -06:00
[GH-ISSUE #1434] app local profiles should be placed before global local profiles #972
Labels
No labels
LTS merge
LTS merge
bug
bug
converted-to-discussion
doc-todo
documentation
duplicate
enhancement
file-transfer
firecfg
firejail-in-firejail
firetools
graphics
help wanted
information_old
installation
invalid
modif
moved
needinfo
networking
notabug
notourbug
old-version
overlayfs
packaging
profile-request
pull-request
question
question_old
removal
runtime-permissions
sandbox-ipc
security
stale
wiki
wiki
wontfix
wordpress
workaround
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/firejail#972
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @ghost on GitHub (Aug 4, 2017).
Original GitHub issue: https://github.com/netblue30/firejail/issues/1434
Currently local profiles are sourced in this precedence:
It means that when user enables some option in
global.localprofile it can't be overridden inark.localprofile. Therefore any global option which breaks one app (example:apparmor) can be overridden only by editingark.profiledirectly which will be overwritten during next update or by creating new profile in~/.config/firejail/ark.profilewhich have to be then maintained solely by user.Basing on the above I propose changing precedence to:
This way user can override any
global.localoption inark.localfile easily. As bothglobal.localandark.localare used only for personal local administration this shouldn't create any security issues.@netblue30 commented on GitHub (Aug 5, 2017):
Sure, it makes sense. Let's wait for @Fred-Barclay and @SpotComms to finish with the current round of cleanups, and I'll write some small script to go through all the files and switch the lines.
@SkewedZeppelin commented on GitHub (Aug 5, 2017):
This is actually already fixed in #1427 since sometime after you opened this.