aboutsummaryrefslogtreecommitdiffstats
path: root/etc/profile-m-z/mutt.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-m-z/mutt.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-m-z/mutt.profile')
-rw-r--r--etc/profile-m-z/mutt.profile120
1 files changed, 60 insertions, 60 deletions
diff --git a/etc/profile-m-z/mutt.profile b/etc/profile-m-z/mutt.profile
index 91606bdfa..debf81659 100644
--- a/etc/profile-m-z/mutt.profile
+++ b/etc/profile-m-z/mutt.profile
@@ -7,36 +7,36 @@ include mutt.local
7# Persistent global definitions 7# Persistent global definitions
8include globals.local 8include globals.local
9 9
10nodeny /var/mail 10noblacklist /var/mail
11nodeny /var/spool/mail 11noblacklist /var/spool/mail
12nodeny ${DOCUMENTS} 12noblacklist ${DOCUMENTS}
13nodeny ${HOME}/.Mail 13noblacklist ${HOME}/.Mail
14nodeny ${HOME}/.bogofilter 14noblacklist ${HOME}/.bogofilter
15nodeny ${HOME}/.cache/mutt 15noblacklist ${HOME}/.cache/mutt
16nodeny ${HOME}/.config/mutt 16noblacklist ${HOME}/.config/mutt
17nodeny ${HOME}/.config/nano 17noblacklist ${HOME}/.config/nano
18nodeny ${HOME}/.elinks 18noblacklist ${HOME}/.elinks
19nodeny ${HOME}/.emacs 19noblacklist ${HOME}/.emacs
20nodeny ${HOME}/.emacs.d 20noblacklist ${HOME}/.emacs.d
21nodeny ${HOME}/.gnupg 21noblacklist ${HOME}/.gnupg
22nodeny ${HOME}/.mail 22noblacklist ${HOME}/.mail
23nodeny ${HOME}/.mailcap 23noblacklist ${HOME}/.mailcap
24nodeny ${HOME}/.msmtprc 24noblacklist ${HOME}/.msmtprc
25nodeny ${HOME}/.mutt 25noblacklist ${HOME}/.mutt
26nodeny ${HOME}/.muttrc 26noblacklist ${HOME}/.muttrc
27nodeny ${HOME}/.nanorc 27noblacklist ${HOME}/.nanorc
28nodeny ${HOME}/.signature 28noblacklist ${HOME}/.signature
29nodeny ${HOME}/.vim 29noblacklist ${HOME}/.vim
30nodeny ${HOME}/.viminfo 30noblacklist ${HOME}/.viminfo
31nodeny ${HOME}/.vimrc 31noblacklist ${HOME}/.vimrc
32nodeny ${HOME}/.w3m 32noblacklist ${HOME}/.w3m
33nodeny ${HOME}/Mail 33noblacklist ${HOME}/Mail
34nodeny ${HOME}/mail 34noblacklist ${HOME}/mail
35nodeny ${HOME}/postponed 35noblacklist ${HOME}/postponed
36nodeny ${HOME}/sent 36noblacklist ${HOME}/sent
37 37
38deny /tmp/.X11-unix 38blacklist /tmp/.X11-unix
39deny ${RUNUSER}/wayland-* 39blacklist ${RUNUSER}/wayland-*
40 40
41# Add the next lines to your mutt.local for oauth.py,S/MIME support. 41# Add the next lines to your mutt.local for oauth.py,S/MIME support.
42#include allow-perl.inc 42#include allow-perl.inc
@@ -75,37 +75,37 @@ mkfile ${HOME}/.nanorc
75mkfile ${HOME}/.signature 75mkfile ${HOME}/.signature
76mkfile ${HOME}/.viminfo 76mkfile ${HOME}/.viminfo
77mkfile ${HOME}/.vimrc 77mkfile ${HOME}/.vimrc
78allow ${DOCUMENTS} 78whitelist ${DOCUMENTS}
79allow ${DOWNLOADS} 79whitelist ${DOWNLOADS}
80allow ${HOME}/.Mail 80whitelist ${HOME}/.Mail
81allow ${HOME}/.bogofilter 81whitelist ${HOME}/.bogofilter
82allow ${HOME}/.cache/mutt 82whitelist ${HOME}/.cache/mutt
83allow ${HOME}/.config/mutt 83whitelist ${HOME}/.config/mutt
84allow ${HOME}/.config/nano 84whitelist ${HOME}/.config/nano
85allow ${HOME}/.elinks 85whitelist ${HOME}/.elinks
86allow ${HOME}/.emacs 86whitelist ${HOME}/.emacs
87allow ${HOME}/.emacs.d 87whitelist ${HOME}/.emacs.d
88allow ${HOME}/.gnupg 88whitelist ${HOME}/.gnupg
89allow ${HOME}/.mail 89whitelist ${HOME}/.mail
90allow ${HOME}/.mailcap 90whitelist ${HOME}/.mailcap
91allow ${HOME}/.msmtprc 91whitelist ${HOME}/.msmtprc
92allow ${HOME}/.mutt 92whitelist ${HOME}/.mutt
93allow ${HOME}/.muttrc 93whitelist ${HOME}/.muttrc
94allow ${HOME}/.nanorc 94whitelist ${HOME}/.nanorc
95allow ${HOME}/.signature 95whitelist ${HOME}/.signature
96allow ${HOME}/.vim 96whitelist ${HOME}/.vim
97allow ${HOME}/.viminfo 97whitelist ${HOME}/.viminfo
98allow ${HOME}/.vimrc 98whitelist ${HOME}/.vimrc
99allow ${HOME}/.w3m 99whitelist ${HOME}/.w3m
100allow ${HOME}/Mail 100whitelist ${HOME}/Mail
101allow ${HOME}/mail 101whitelist ${HOME}/mail
102allow ${HOME}/postponed 102whitelist ${HOME}/postponed
103allow ${HOME}/sent 103whitelist ${HOME}/sent
104allow /usr/share/gnupg 104whitelist /usr/share/gnupg
105allow /usr/share/gnupg2 105whitelist /usr/share/gnupg2
106allow /usr/share/mutt 106whitelist /usr/share/mutt
107allow /var/mail 107whitelist /var/mail
108allow /var/spool/mail 108whitelist /var/spool/mail
109include whitelist-common.inc 109include whitelist-common.inc
110include whitelist-runuser-common.inc 110include whitelist-runuser-common.inc
111include whitelist-usr-share-common.inc 111include whitelist-usr-share-common.inc