From adaf3d6415865cd288fea8afb91d04a524ccb80c Mon Sep 17 00:00:00 2001 From: Joseph Nuthalapati Date: Wed, 27 Mar 2019 10:57:16 +0530 Subject: [PATCH] letsencrypt: Call letsencrypt manage_hooks with correct arguments The `--modules` argument in letsencrypt toggle_hooks command is passed a list of plinth modules as a single space-separated string. They should instead be passed as a sequence of individual arguments. Fixes #1520 Signed-off-by: Joseph Nuthalapati Reviewed-by: Sunil Mohan Adapa --- plinth/modules/letsencrypt/views.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plinth/modules/letsencrypt/views.py b/plinth/modules/letsencrypt/views.py index 466a67a0e..54740fad7 100644 --- a/plinth/modules/letsencrypt/views.py +++ b/plinth/modules/letsencrypt/views.py @@ -156,7 +156,7 @@ def toggle_module(request, domain, module): le_arguments = ['manage_hooks', 'enable'] if not enabled_modules == []: - le_arguments.extend(['--modules', ' '.join(enabled_modules)]) + le_arguments.extend(['--modules'] + enabled_modules) try: actions.superuser_run(module, module_args)