Skip to content

Commit 2fc23db

Browse files
author
Marius Burkard
committed
- limit domains for LE cert to 100, fixes #4789
1 parent 39f51a4 commit 2fc23db

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

server/lib/classes/letsencrypt.inc.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -250,6 +250,12 @@ public function request_certificates($data, $server_type = 'apache') {
250250
unset($le_domains);
251251
@unlink('/usr/local/ispconfig/interface/acme/.well-known/acme-challenge/' . $le_rnd_file);
252252

253+
$le_domain_count = count($temp_domains);
254+
if($le_domain_count > 100) {
255+
$temp_domains = array_splice($temp_domains, 0, 100);
256+
$app->log("There were " . $le_domain_count . " domains in the domain list. LE only supports 100, so we strip the rest.", LOGLEVEL_WARN);
257+
}
258+
253259
// generate cli format
254260
foreach($temp_domains as $temp_domain) {
255261
$cli_domain_arg .= (string) " --domains " . $temp_domain;

0 commit comments

Comments
 (0)