From 529bbf77df9194d73ce888972db44a0377eeafc8 Mon Sep 17 00:00:00 2001 From: Sunil Mohan Adapa Date: Wed, 13 May 2026 13:36:25 -0700 Subject: [PATCH] bepasty: Don't remove old system user and group As removing a system user and group is considered a bad practice. Old unused system accounts are mostly harmless. Tests: - Bepasty functional tests pass. - Installing bepasty and uploading a file works. Signed-off-by: Sunil Mohan Adapa --- plinth/modules/bepasty/privileged.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/plinth/modules/bepasty/privileged.py b/plinth/modules/bepasty/privileged.py index 80bdcb439..b89833e6a 100644 --- a/plinth/modules/bepasty/privileged.py +++ b/plinth/modules/bepasty/privileged.py @@ -91,8 +91,6 @@ def setup(domain_name: str): # Migrate from old bepasty:bepasty ownership to root:root shutil.chown(CONF_FILE, user='root', group='root') - action_utils.run(['deluser', 'bepasty'], check=False) - action_utils.run(['delgroup', 'bepasty'], check=False) @privileged @@ -151,6 +149,6 @@ def _generate_password(): @privileged def uninstall(): - """Remove bepasty user, group and data.""" + """Remove data and configuration file.""" shutil.rmtree(DATA_DIR, ignore_errors=True) CONF_FILE.unlink(missing_ok=True)