diff options
Diffstat (limited to 'modules-available/webinterface/inc/acme.inc.php')
| -rw-r--r-- | modules-available/webinterface/inc/acme.inc.php | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/modules-available/webinterface/inc/acme.inc.php b/modules-available/webinterface/inc/acme.inc.php index 446c0e45..3f5e76a0 100644 --- a/modules-available/webinterface/inc/acme.inc.php +++ b/modules-available/webinterface/inc/acme.inc.php @@ -94,13 +94,24 @@ class Acme { if (!Taskmanager::isFinished($task)) return; - if (Taskmanager::isFailed($task)) { + $otherError = false; + if (isset($task['data']['error'])) { + // This should never happen, so make it an error + if (strpos($task['data']['error'], " is not an issued domain, skipping.") !== false) { + $otherError = true; + } + } + if (Taskmanager::isFailed($task) || $otherError) { if (($args['user'] ?? null) === null) { EventLog::warning('Automatic ACME renewal of HTTPS certificate failed', print_r($task, true)); } Property::set(self::PROP_ERROR, $task['data']['error'] ?? 'Unknown error'); } else { - EventLog::info('ACME issue/renewal of HTTPS certificate by ' . ($args['user'] ?? 'automatic cronjob')); + // If the cronjob called us and there is nothing to do, suppress the event log entry + if (($args['user'] ?? null) !== null + || strpos($task['data']['error'] ?? '', 'Skipping. Next renewal time is: ') === false) { + EventLog::info('ACME issue/renewal of HTTPS certificate by ' . ($args['user'] ?? 'automatic cronjob'), print_r($task, true)); + } Property::set(self::PROP_ERROR, false); } } |