maketank/Froxlor
2
0
Fork 0
Code Pull Requests Releases Activity

do not create vhost content for ssl-vhosts that do not have a ssl-certificate specified, fixes #1583

Browse Source 
Signed-off-by: Michael Kaufmann (d00p) <d00p@froxlor.org>
This commit is contained in:
Michael Kaufmann (d00p)
2016-01-15 09:40:59 +01:00
parent f2c3b3f165
commit 29494b71fa
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
scripts/jobs/cron_tasks.inc.http.10.apache.php
View File

@@ -818,6 +818,13 @@ class apache extends HttpConfigBase {
$vhost_content .= ' SSLCertificateChainFile ' . makeCorrectFile($domain['ssl_cert_chainfile']) . "\n";
}
}
else
{
// if there is no cert-file specified but we are generating a ssl-vhost,
// we should return an empty string because this vhost would suck dick, ref #1583
$this->logger->logAction(CRON_ACTION, LOG_ERROR, $domain['domain'] . ' :: empty certificate file! Cannot create ssl-directives');
return '# no ssl-certificate was specified for this domain, therefore no explicit vhost is being generated';
}
}
if (preg_match('/^https?\:\/\//', $domain['documentroot'])) {