aboutsummaryrefslogtreecommitdiffstats
path: root/etc/profile-m-z/neomutt.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/neomutt.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/neomutt.profile')
-rw-r--r--etc/profile-m-z/neomutt.profile128
1 files changed, 64 insertions, 64 deletions
diff --git a/etc/profile-m-z/neomutt.profile b/etc/profile-m-z/neomutt.profile
index 939d6f111..fafa129e4 100644
--- a/etc/profile-m-z/neomutt.profile
+++ b/etc/profile-m-z/neomutt.profile
@@ -7,38 +7,38 @@ include neomutt.local
7# Persistent global definitions 7# Persistent global definitions
8include globals.local 8include globals.local
9 9
10nodeny ${DOCUMENTS} 10noblacklist ${DOCUMENTS}
11nodeny ${HOME}/.Mail 11noblacklist ${HOME}/.Mail
12nodeny ${HOME}/.bogofilter 12noblacklist ${HOME}/.bogofilter
13nodeny ${HOME}/.config/mutt 13noblacklist ${HOME}/.config/mutt
14nodeny ${HOME}/.config/nano 14noblacklist ${HOME}/.config/nano
15nodeny ${HOME}/.config/neomutt 15noblacklist ${HOME}/.config/neomutt
16nodeny ${HOME}/.elinks 16noblacklist ${HOME}/.elinks
17nodeny ${HOME}/.emacs 17noblacklist ${HOME}/.emacs
18nodeny ${HOME}/.emacs.d 18noblacklist ${HOME}/.emacs.d
19nodeny ${HOME}/.gnupg 19noblacklist ${HOME}/.gnupg
20nodeny ${HOME}/.mail 20noblacklist ${HOME}/.mail
21nodeny ${HOME}/.mailcap 21noblacklist ${HOME}/.mailcap
22nodeny ${HOME}/.msmtprc 22noblacklist ${HOME}/.msmtprc
23nodeny ${HOME}/.mutt 23noblacklist ${HOME}/.mutt
24nodeny ${HOME}/.muttrc 24noblacklist ${HOME}/.muttrc
25nodeny ${HOME}/.nanorc 25noblacklist ${HOME}/.nanorc
26nodeny ${HOME}/.neomutt 26noblacklist ${HOME}/.neomutt
27nodeny ${HOME}/.neomuttrc 27noblacklist ${HOME}/.neomuttrc
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
37nodeny /var/mail 37noblacklist /var/mail
38nodeny /var/spool/mail 38noblacklist /var/spool/mail
39 39
40deny /tmp/.X11-unix 40blacklist /tmp/.X11-unix
41deny ${RUNUSER}/wayland-* 41blacklist ${RUNUSER}/wayland-*
42 42
43include allow-lua.inc 43include allow-lua.inc
44 44
@@ -76,39 +76,39 @@ mkfile ${HOME}/.neomuttrc
76mkfile ${HOME}/.signature 76mkfile ${HOME}/.signature
77mkfile ${HOME}/.viminfo 77mkfile ${HOME}/.viminfo
78mkfile ${HOME}/.vimrc 78mkfile ${HOME}/.vimrc
79allow ${DOCUMENTS} 79whitelist ${DOCUMENTS}
80allow ${DOWNLOADS} 80whitelist ${DOWNLOADS}
81allow ${HOME}/.Mail 81whitelist ${HOME}/.Mail
82allow ${HOME}/.bogofilter 82whitelist ${HOME}/.bogofilter
83allow ${HOME}/.config/mutt 83whitelist ${HOME}/.config/mutt
84allow ${HOME}/.config/nano 84whitelist ${HOME}/.config/nano
85allow ${HOME}/.config/neomutt 85whitelist ${HOME}/.config/neomutt
86allow ${HOME}/.elinks 86whitelist ${HOME}/.elinks
87allow ${HOME}/.emacs 87whitelist ${HOME}/.emacs
88allow ${HOME}/.emacs.d 88whitelist ${HOME}/.emacs.d
89allow ${HOME}/.gnupg 89whitelist ${HOME}/.gnupg
90allow ${HOME}/.mail 90whitelist ${HOME}/.mail
91allow ${HOME}/.mailcap 91whitelist ${HOME}/.mailcap
92allow ${HOME}/.msmtprc 92whitelist ${HOME}/.msmtprc
93allow ${HOME}/.mutt 93whitelist ${HOME}/.mutt
94allow ${HOME}/.muttrc 94whitelist ${HOME}/.muttrc
95allow ${HOME}/.nanorc 95whitelist ${HOME}/.nanorc
96allow ${HOME}/.neomutt 96whitelist ${HOME}/.neomutt
97allow ${HOME}/.neomuttrc 97whitelist ${HOME}/.neomuttrc
98allow ${HOME}/.signature 98whitelist ${HOME}/.signature
99allow ${HOME}/.vim 99whitelist ${HOME}/.vim
100allow ${HOME}/.viminfo 100whitelist ${HOME}/.viminfo
101allow ${HOME}/.vimrc 101whitelist ${HOME}/.vimrc
102allow ${HOME}/.w3m 102whitelist ${HOME}/.w3m
103allow ${HOME}/Mail 103whitelist ${HOME}/Mail
104allow ${HOME}/mail 104whitelist ${HOME}/mail
105allow ${HOME}/postponed 105whitelist ${HOME}/postponed
106allow ${HOME}/sent 106whitelist ${HOME}/sent
107allow /usr/share/gnupg 107whitelist /usr/share/gnupg
108allow /usr/share/gnupg2 108whitelist /usr/share/gnupg2
109allow /usr/share/neomutt 109whitelist /usr/share/neomutt
110allow /var/mail 110whitelist /var/mail
111allow /var/spool/mail 111whitelist /var/spool/mail
112include whitelist-common.inc 112include whitelist-common.inc
113include whitelist-runuser-common.inc 113include whitelist-runuser-common.inc
114include whitelist-usr-share-common.inc 114include whitelist-usr-share-common.inc