From 4c5344dfa06b67d1f6d3ffa3e89f99d82f25ea9b Mon Sep 17 00:00:00 2001 From: Sunil Mohan Adapa Date: Mon, 24 Oct 2022 12:25:58 -0700 Subject: [PATCH] letsencrypt: Fix regression with comparing certificate Closes: #2295. _assert_managed_path() expects pathlib.Path. Due to a typo, a string is being sent instead. Signed-off-by: Sunil Mohan Adapa Reviewed-by: James Valleroy --- plinth/modules/letsencrypt/privileged.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plinth/modules/letsencrypt/privileged.py b/plinth/modules/letsencrypt/privileged.py index d77e9e29a..516957bd3 100644 --- a/plinth/modules/letsencrypt/privileged.py +++ b/plinth/modules/letsencrypt/privileged.py @@ -272,8 +272,8 @@ def compare_certificate(managing_app: str, source_private_key: str, private_key_path = pathlib.Path(private_key) certificate_path = pathlib.Path(certificate) - _assert_managed_path(managing_app, private_key) - _assert_managed_path(managing_app, certificate) + _assert_managed_path(managing_app, private_key_path) + _assert_managed_path(managing_app, certificate_path) result = False try: