From 2fc23db2abf2625d5fc514c8af969a06891bb789 Mon Sep 17 00:00:00 2001 From: Marius Burkard Date: Mon, 18 Sep 2017 15:38:03 +0200 Subject: [PATCH] - limit domains for LE cert to 100, fixes #4789 --- server/lib/classes/letsencrypt.inc.php | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/server/lib/classes/letsencrypt.inc.php b/server/lib/classes/letsencrypt.inc.php index 28a4ec011..f27b57804 100644 --- a/server/lib/classes/letsencrypt.inc.php +++ b/server/lib/classes/letsencrypt.inc.php @@ -250,6 +250,12 @@ class letsencrypt { unset($le_domains); @unlink('/usr/local/ispconfig/interface/acme/.well-known/acme-challenge/' . $le_rnd_file); + $le_domain_count = count($temp_domains); + if($le_domain_count > 100) { + $temp_domains = array_splice($temp_domains, 0, 100); + $app->log("There were " . $le_domain_count . " domains in the domain list. LE only supports 100, so we strip the rest.", LOGLEVEL_WARN); + } + // generate cli format foreach($temp_domains as $temp_domain) { $cli_domain_arg .= (string) " --domains " . $temp_domain; -- GitLab