From 631e10c09e7625018088c16fb128a4443468267c Mon Sep 17 00:00:00 2001 From: Veiko Aasa Date: Tue, 2 Mar 2021 18:20:57 +0200 Subject: [PATCH] deluge: Fix daemon user not in freedombox-share group after installation Add the debian-deluged user to the freedombox-share group *before* running app setup. The app setup process restarts the deluged daemon after which the process has correct group ownerships. Tests done in Debian stable and testing dev containers: - After installing the deluge app, checked from /proc/--process-id--/status that the deluged daemon process has the freedombox-share group. Signed-off-by: Veiko Aasa Reviewed-by: Sunil Mohan Adapa --- plinth/modules/deluge/__init__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plinth/modules/deluge/__init__.py b/plinth/modules/deluge/__init__.py index 15bc3962b..3e82bd30d 100644 --- a/plinth/modules/deluge/__init__.py +++ b/plinth/modules/deluge/__init__.py @@ -98,6 +98,6 @@ class DelugeApp(app_module.App): def setup(helper, old_version=None): """Install and configure the module.""" helper.install(managed_packages) - helper.call('post', actions.superuser_run, 'deluge', ['setup']) add_user_to_share_group(SYSTEM_USER) + helper.call('post', actions.superuser_run, 'deluge', ['setup']) helper.call('post', app.enable)