aboutsummaryrefslogtreecommitdiffstats
path: root/etc/profile-a-l/firefox-common-addons.profile
diff options
context:
space:
mode:
authorLibravatar Kelvin M. Klann <kmk3.code@protonmail.com>2021-07-18 20:39:14 -0300
committerLibravatar Kelvin M. Klann <kmk3.code@protonmail.com>2021-07-18 20:39:14 -0300
commitf43382f1e9707b4fd5e63c7bfe881912aa4ee994 (patch)
tree499639bb962c8b071b153dcdad1b42af8286521d /etc/profile-a-l/firefox-common-addons.profile
parentAdd MS Edge Beta profile (diff)
downloadfirejail-f43382f1e9707b4fd5e63c7bfe881912aa4ee994.tar.gz
firejail-f43382f1e9707b4fd5e63c7bfe881912aa4ee994.tar.zst
firejail-f43382f1e9707b4fd5e63c7bfe881912aa4ee994.zip
Revert "move whitelist/blacklist to allow/deny"
This reverts commit fe0f975f447d59977d90c3226cc8c623b31b20b3. Note: This only reverts the changes from etc. The 4 aliases introduced on commit 45f2ba544 are mere, well, aliases. That is, they fail to address the different usability problems discussed on [#3447][3447] and in fact only make things more confusing (as has already been mentioned on [this][4379] and later comments). The main reason is that the aliases do not meaningfully map to the original commands. For example, the commands from each pair below seem like they would do the exact same thing: * `allow` and `nodeny` * `deny` and `noallow` Additionally, if these aliases are not the final commands, but only a test/work-in-progress, then keeping the wide-scale search/replace changes made on commit fe0f975f4 would only serve to cause confusion, as users of firejail-git, contributors and downstream projects might start changing the commands used on their profiles, only to later have to change them again, potentially to completely different commands. The sooner this is undone the better, as (besides the above reasons) the more profile changes there are between the original commit and the revert, the harder it is to e.g.: `git diff` versions of files across the following revision ranges: before the commit, after the commit but before the revert and after the revert. Note: This is still the case even if a commit is [ignored by `git blame`][4390]. So let us revert fe0f975f4 and only reapply similar large-scale changes once we have discussed and settled on better commands. How the revert was applied: Despite using the auto-generated message from `git revert`, to ensure correctness and to avoid conflicts the changes were reverted in different steps: Firstly, revert the files which can be safely reverted directly ("filestorevert"): # Find out which files have been changed on fe0f975f44, but have not # been changed afterwards and list them on "filestorevert" git show --pretty='' --name-only fe0f975f44 -- etc | LC_ALL=C sort >allfiles git diff --name-only fe0f975f44..master -- etc | LC_ALL=C sort >filestoignore comm -2 -3 allfiles filestoignore >filestorevert # Note: There are 3 extra files on filestoignore because they were # added after commit fe0f975f44 wc -l allfiles filestoignore filestorevert | head -n 3 # 797 allfiles # 8 filestoignore # 792 filestorevert # Automatically revert files in "filestorevert" # See https://stackoverflow.com/a/23401018/10095231 tr '\n' '\000' <filestorevert | xargs -0 git show fe0f975f44 -- | git apply --reverse printf 'Total files reverted:\n' git diff --name-only | wc -l # 792 Secondly, do some search/replace on the rest: tr '\n' '\000' <filestoignore | xargs -0 sed -i.bak \ -e 's/allow /whitelist /' -e 's/noallow /nowhitelist /' \ -e 's/deny /blacklist /' -e 's/nodeny /noblacklist /' \ -e 's/deny-nolog /blacklist-nolog /' find etc -name '*.bak' -print0 | xargs -0 rm Thirdly, verify the result. The following command shows the difference between all the changes in etc from before fe0f975f44 and this commit (inclusive): git diff fe0f975f44~1 -- etc From the output, it looks like all alias changes are fully reverted and that the other changes to etc (from after fe0f975f44) remain, so the revert seems to be done correctly. [3447]: https://github.com/netblue30/firejail/issues/3447 [4379]: https://github.com/netblue30/firejail/issues/4379#issuecomment-876460222 [4390]: https://github.com/netblue30/firejail/issues/4390
Diffstat (limited to 'etc/profile-a-l/firefox-common-addons.profile')
-rw-r--r--etc/profile-a-l/firefox-common-addons.profile130
1 files changed, 65 insertions, 65 deletions
diff --git a/etc/profile-a-l/firefox-common-addons.profile b/etc/profile-a-l/firefox-common-addons.profile
index 7e2e8760d..d282f9a60 100644
--- a/etc/profile-a-l/firefox-common-addons.profile
+++ b/etc/profile-a-l/firefox-common-addons.profile
@@ -5,74 +5,74 @@ include firefox-common-addons.local
5ignore include whitelist-runuser-common.inc 5ignore include whitelist-runuser-common.inc
6ignore private-cache 6ignore private-cache
7 7
8nodeny ${HOME}/.cache/youtube-dl 8noblacklist ${HOME}/.cache/youtube-dl
9nodeny ${HOME}/.config/kgetrc 9noblacklist ${HOME}/.config/kgetrc
10nodeny ${HOME}/.config/mpv 10noblacklist ${HOME}/.config/mpv
11nodeny ${HOME}/.config/okularpartrc 11noblacklist ${HOME}/.config/okularpartrc
12nodeny ${HOME}/.config/okularrc 12noblacklist ${HOME}/.config/okularrc
13nodeny ${HOME}/.config/qpdfview 13noblacklist ${HOME}/.config/qpdfview
14nodeny ${HOME}/.config/youtube-dl 14noblacklist ${HOME}/.config/youtube-dl
15nodeny ${HOME}/.kde/share/apps/kget 15noblacklist ${HOME}/.kde/share/apps/kget
16nodeny ${HOME}/.kde/share/apps/okular 16noblacklist ${HOME}/.kde/share/apps/okular
17nodeny ${HOME}/.kde/share/config/kgetrc 17noblacklist ${HOME}/.kde/share/config/kgetrc
18nodeny ${HOME}/.kde/share/config/okularpartrc 18noblacklist ${HOME}/.kde/share/config/okularpartrc
19nodeny ${HOME}/.kde/share/config/okularrc 19noblacklist ${HOME}/.kde/share/config/okularrc
20nodeny ${HOME}/.kde4/share/apps/kget 20noblacklist ${HOME}/.kde4/share/apps/kget
21nodeny ${HOME}/.kde4/share/apps/okular 21noblacklist ${HOME}/.kde4/share/apps/okular
22nodeny ${HOME}/.kde4/share/config/kgetrc 22noblacklist ${HOME}/.kde4/share/config/kgetrc
23nodeny ${HOME}/.kde4/share/config/okularpartrc 23noblacklist ${HOME}/.kde4/share/config/okularpartrc
24nodeny ${HOME}/.kde4/share/config/okularrc 24noblacklist ${HOME}/.kde4/share/config/okularrc
25nodeny ${HOME}/.local/share/kget 25noblacklist ${HOME}/.local/share/kget
26nodeny ${HOME}/.local/share/kxmlgui5/okular 26noblacklist ${HOME}/.local/share/kxmlgui5/okular
27nodeny ${HOME}/.local/share/okular 27noblacklist ${HOME}/.local/share/okular
28nodeny ${HOME}/.local/share/qpdfview 28noblacklist ${HOME}/.local/share/qpdfview
29nodeny ${HOME}/.netrc 29noblacklist ${HOME}/.netrc
30 30
31allow ${HOME}/.cache/gnome-mplayer/plugin 31whitelist ${HOME}/.cache/gnome-mplayer/plugin
32allow ${HOME}/.cache/youtube-dl/youtube-sigfuncs 32whitelist ${HOME}/.cache/youtube-dl/youtube-sigfuncs
33allow ${HOME}/.config/gnome-mplayer 33whitelist ${HOME}/.config/gnome-mplayer
34allow ${HOME}/.config/kgetrc 34whitelist ${HOME}/.config/kgetrc
35allow ${HOME}/.config/mpv 35whitelist ${HOME}/.config/mpv
36allow ${HOME}/.config/okularpartrc 36whitelist ${HOME}/.config/okularpartrc
37allow ${HOME}/.config/okularrc 37whitelist ${HOME}/.config/okularrc
38allow ${HOME}/.config/pipelight-silverlight5.1 38whitelist ${HOME}/.config/pipelight-silverlight5.1
39allow ${HOME}/.config/pipelight-widevine 39whitelist ${HOME}/.config/pipelight-widevine
40allow ${HOME}/.config/qpdfview 40whitelist ${HOME}/.config/qpdfview
41allow ${HOME}/.config/youtube-dl 41whitelist ${HOME}/.config/youtube-dl
42allow ${HOME}/.kde/share/apps/kget 42whitelist ${HOME}/.kde/share/apps/kget
43allow ${HOME}/.kde/share/apps/okular 43whitelist ${HOME}/.kde/share/apps/okular
44allow ${HOME}/.kde/share/config/kgetrc 44whitelist ${HOME}/.kde/share/config/kgetrc
45allow ${HOME}/.kde/share/config/okularpartrc 45whitelist ${HOME}/.kde/share/config/okularpartrc
46allow ${HOME}/.kde/share/config/okularrc 46whitelist ${HOME}/.kde/share/config/okularrc
47allow ${HOME}/.kde4/share/apps/kget 47whitelist ${HOME}/.kde4/share/apps/kget
48allow ${HOME}/.kde4/share/apps/okular 48whitelist ${HOME}/.kde4/share/apps/okular
49allow ${HOME}/.kde4/share/config/kgetrc 49whitelist ${HOME}/.kde4/share/config/kgetrc
50allow ${HOME}/.kde4/share/config/okularpartrc 50whitelist ${HOME}/.kde4/share/config/okularpartrc
51allow ${HOME}/.kde4/share/config/okularrc 51whitelist ${HOME}/.kde4/share/config/okularrc
52allow ${HOME}/.keysnail.js 52whitelist ${HOME}/.keysnail.js
53allow ${HOME}/.lastpass 53whitelist ${HOME}/.lastpass
54allow ${HOME}/.local/share/kget 54whitelist ${HOME}/.local/share/kget
55allow ${HOME}/.local/share/kxmlgui5/okular 55whitelist ${HOME}/.local/share/kxmlgui5/okular
56allow ${HOME}/.local/share/okular 56whitelist ${HOME}/.local/share/okular
57allow ${HOME}/.local/share/qpdfview 57whitelist ${HOME}/.local/share/qpdfview
58allow ${HOME}/.local/share/tridactyl 58whitelist ${HOME}/.local/share/tridactyl
59allow ${HOME}/.netrc 59whitelist ${HOME}/.netrc
60allow ${HOME}/.pentadactyl 60whitelist ${HOME}/.pentadactyl
61allow ${HOME}/.pentadactylrc 61whitelist ${HOME}/.pentadactylrc
62allow ${HOME}/.tridactylrc 62whitelist ${HOME}/.tridactylrc
63allow ${HOME}/.vimperator 63whitelist ${HOME}/.vimperator
64allow ${HOME}/.vimperatorrc 64whitelist ${HOME}/.vimperatorrc
65allow ${HOME}/.wine-pipelight 65whitelist ${HOME}/.wine-pipelight
66allow ${HOME}/.wine-pipelight64 66whitelist ${HOME}/.wine-pipelight64
67allow ${HOME}/.zotero 67whitelist ${HOME}/.zotero
68allow ${HOME}/dwhelper 68whitelist ${HOME}/dwhelper
69allow /usr/share/lua 69whitelist /usr/share/lua
70allow /usr/share/lua* 70whitelist /usr/share/lua*
71allow /usr/share/vulkan 71whitelist /usr/share/vulkan
72 72
73# GNOME Shell integration (chrome-gnome-shell) needs dbus and python 73# GNOME Shell integration (chrome-gnome-shell) needs dbus and python
74nodeny ${HOME}/.local/share/gnome-shell 74noblacklist ${HOME}/.local/share/gnome-shell
75allow ${HOME}/.local/share/gnome-shell 75whitelist ${HOME}/.local/share/gnome-shell
76dbus-user.talk ca.desrt.dconf 76dbus-user.talk ca.desrt.dconf
77dbus-user.talk org.gnome.ChromeGnomeShell 77dbus-user.talk org.gnome.ChromeGnomeShell
78dbus-user.talk org.gnome.Shell 78dbus-user.talk org.gnome.Shell