Skip to content

Commit

Permalink
do not remove Let's Encrypt token when self-check fails but rather gi…
Browse files Browse the repository at this point in the history
…ve out the information as warning. The self-check fails for many users due to different local configurations and might not always be correct; fixes #480

Signed-off-by: Michael Kaufmann (d00p) <d00p@froxlor.org>
  • Loading branch information
d00p committed Nov 27, 2017
1 parent e725b48 commit 5540b02
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions lib/classes/ssl/class.lescript.php
Expand Up @@ -236,9 +236,7 @@ public function signDomains(array $domains, $domainkey = null, $csr = null)
} else {
$errmsg = "";
}
@unlink($tokenPath);
$this->logger->logAction(CRON_ACTION, LOG_ERR, "letsencrypt Please check $uri - token not available" . $errmsg);
continue;
$this->logger->logAction(CRON_ACTION, LOG_WARNING, "[Lets Encrypt self-check] Please check $uri - token seems to be not available. This is just a simple self-check, it might be wrong but consider using this information when Let's Encrypt fails to issue a certificate" . $errmsg);
}

$this->log("Sending request to challenge");
Expand Down

0 comments on commit 5540b02

Please sign in to comment.