nik/FreedomBox
1
0
Fork 0
mirror of https://github.com/freedombox/FreedomBox.git synced 2026-07-29 12:09:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

tor, torproxy: Avoid error if defaults file missing during uninstall

Browse Source 
See:
https://discuss.freedombox.org/t/error-when-uninstalling-tor-and-tor-proxy/4295

sunil: Use pathlib.Path.unlink()

Tests:

- Install Tor and Tor Proxy. Uninstall both. Uninstall is succesful.

- Install Tor and Tor Proxy. Manually delete
  /var/run/tor-instances/*.defaults. Uninstall both apps. Uninstall is
  successful. A warning is printed in the logs.

Signed-off-by: James Valleroy <jvalleroy@mailbox.org>
Signed-off-by: Sunil Mohan Adapa <sunil@medhas.org>
Reviewed-by: Sunil Mohan Adapa <sunil@medhas.org>
This commit is contained in:
James Valleroy 2026-07-11 15:39:06 -04:00 committed by Sunil Mohan Adapa
parent fce97a871a
commit 10069c4e5d
No known key found for this signature in database
GPG Key ID: 43EA1CFF0AA7C5F2
2 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
plinth/modules/tor/privileged.py
View File

@ -494,5 +494,7 @@ def uninstall():
for directory in directories:
shutil.rmtree(directory, ignore_errors=True)
os.unlink(f'/var/run/tor-instances/{INSTANCE_NAME}.defaults')
defaults_file = f'/var/run/tor-instances/{INSTANCE_NAME}.defaults'
pathlib.Path(defaults_file).unlink(missing_ok=True)
action_utils.service_unmask(SERVICE_NAME)

6
plinth/modules/torproxy/privileged.py
View File

@ -2,7 +2,7 @@
"""Configure Tor Proxy service."""
import logging
import os
import pathlib
import shutil
from typing import Any
@ -190,5 +190,7 @@ def uninstall():
for directory in directories:
shutil.rmtree(directory, ignore_errors=True)
os.unlink(f'/var/run/tor-instances/{INSTANCE_NAME}.defaults')
defaults_file = f'/var/run/tor-instances/{INSTANCE_NAME}.defaults'
pathlib.Path(defaults_file).unlink(missing_ok=True)
action_utils.service_unmask(SERVICE_NAME)