From 58c6ca95b18948d9d1099ba7fb1300d3b33b0fcc Mon Sep 17 00:00:00 2001 From: dpeca Date: Mon, 10 Dec 2018 14:15:46 +0100 Subject: [PATCH] Fix for suspended domain checking while LE renewing Logic should be inversed... If you grep for suspended=no, then variable is NOT empty if it's NOT suspended Which means it's empty only if it's suspended. So we must grep for suspended=yes if we consider not-empty result as suspended domain. @skullwritter This bug totally brakes LE renewing :) --- bin/v-update-letsencrypt-ssl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bin/v-update-letsencrypt-ssl b/bin/v-update-letsencrypt-ssl index 6e26d7e5e1..dd1f3b7cc2 100755 --- a/bin/v-update-letsencrypt-ssl +++ b/bin/v-update-letsencrypt-ssl @@ -32,7 +32,7 @@ for user in $users; do lecounter=0 for domain in $(search_objects 'web' 'LETSENCRYPT' 'yes' 'DOMAIN'); do # Working on Web domain check - if is suspended - webSuspended=$(grep "DOMAIN='$domain'" $USER_DATA/web.conf |grep "SUSPENDED='no") + webSuspended=$(grep "DOMAIN='$domain'" $USER_DATA/web.conf |grep "SUSPENDED='yes") if [ ! -z "$webSuspended" ]; then continue; fi;