-
Notifications
You must be signed in to change notification settings - Fork 557
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
Fix for KeePassXC plugin #3984
Fix for KeePassXC plugin #3984
Conversation
KeePassXC changed the socket name. keepassxreboot/keepassxc@a145bf9 Keep also old socket name in whitelist for back compatibility
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.
Since it is a whitelist
, it shoud go to the whitelist block(s) above (line 12-23).
Co-authored-by: rusty-snake <[email protected]>
and since it is a Add-On related thing, it should go in firefox-common-addons.inc. There we conflict with |
Genau! :-) |
Very strange.
|
Sorry, I don't fully get you. firefox-common-addons.inc is opt-in because it has wide relaxations and is used by a minority, if this is where you hang. |
So, that's the reason why I cannot see the parse of firefox-common-addons.inc while starting from the terminal.
P.S. firefox.profile and keepassxc.profile are there to override the default from .deb (so I'm sure to use the very last from the git) |
(For cross-reference, this PR stems from the following issue: #3952) |
Tried again this morning without good results. For the test, I deleted firefox.local and used the updated version for all firefox.profiles,
|
Moved in the right place as in template https://github.com/netblue30/firejail/blob/master/etc/templates/profile.template
What about this profile? |
My worry is that I don't understand why firefox-common-addons.inc doesn't work. |
Mhhh, clear. One question: is the new named bus from keepassxc already included somewhere? if not, why firefox-common-addons should allow it? ...maybe I'm missing something |
Merged, thanks! |
KeePassXC changed the socket name.
keepassxreboot/keepassxc@a145bf9
Keep also old socket name in whitelist for back compatibility