Hello Erik,
Thanks for the patch, but doesn’t it look a little bit over-engineered?
On 30 Nov 2023, at 07:56, Erik Kapfer erik.kapfer@ipfire.org wrote:
Signed-off-by: Erik Kapfer erik.kapfer@ipfire.org
config/rootfiles/core/182/update.sh | 11 +++++++++++ 1 file changed, 11 insertions(+)
diff --git a/config/rootfiles/core/182/update.sh b/config/rootfiles/core/182/update.sh index 4416e73f4..cb7d853ec 100644 --- a/config/rootfiles/core/182/update.sh +++ b/config/rootfiles/core/182/update.sh @@ -91,6 +91,17 @@ if grep -q "ENABLED=on" /var/ipfire/vpn/settings; then /etc/rc.d/init.d/ipsec start fi
+# Change deprecated option in tor configuration file if in usage +if grep -q 'ExitNode ' /var/ipfire/tor/torrc; then
- if pgrep tor >/dev/null; then
- /usr/local/bin/torctrl stop >/dev/null
- sed -i 's/ExitNode /ExitNodes /g' /var/ipfire/tor/torrc
- /usr/local/bin/torctrl restart >/dev/null
- fi
+else
- sed -i 's/ExitNode /ExitNodes /g' /var/ipfire/tor/torrc
You can always run this command without checking for “ExitNode “ because, because it wouldn’t break anything.
And I think that stopping Tor feels a little bit complicated, or does it actually overwrite its own configuration file?
-Michael
+fi
# Rebuild initial ramdisks dracut --regenerate-all --force KVER="xxxKVERxxx" -- 2.42.0