Skip to content

Commit 29d7a4e

Browse files
author
Till Brehm
committed
Merge branch 'certificate_exists' into 'develop'
remove unneeded "certificate exists" message Closes #5757 See merge request ispconfig/ispconfig3!1206
2 parents a8273bc + c28faf1 commit 29d7a4e

File tree

2 files changed

+20
-12
lines changed

2 files changed

+20
-12
lines changed

install/install.php

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -545,6 +545,7 @@
545545
}
546546

547547
//** Configure ISPConfig :-)
548+
$issue_asked = false;
548549
$issue_tried = false;
549550
$install_ispconfig_interface_default = ($conf['mysql']['master_slave_setup'] == 'y')?'n':'y';
550551
if($install_mode == 'standard' || strtolower($inst->simple_query('Install ISPConfig Web Interface', array('y', 'n'), $install_ispconfig_interface_default,'install_ispconfig_web_interface')) == 'y') {
@@ -574,19 +575,22 @@
574575
$inst->make_ispconfig_ssl_cert();
575576
$issue_tried = true;
576577
}
578+
$issue_asked = true;
577579
$inst->install_ispconfig_interface = true;
578580

579581
} else {
580582
$inst->install_ispconfig_interface = false;
581583
}
582584

583585
// Create SSL certs for non-webserver(s)?
584-
if(!file_exists('/usr/local/ispconfig/interface/ssl/ispserver.crt')) {
585-
if(!$issue_tried && strtolower($inst->simple_query('Do you want to create SSL certs for your server?', array('y', 'n'), 'y','create_ssl_server_certs')) == 'y') {
586-
$inst->make_ispconfig_ssl_cert();
587-
}
588-
} else {
589-
swriteln('Certificate exists. Not creating a new one.');
586+
if(!$issue_asked) {
587+
if(!file_exists('/usr/local/ispconfig/interface/ssl/ispserver.crt')) {
588+
if(!$issue_tried && strtolower($inst->simple_query('Do you want to create SSL certs for your server?', array('y', 'n'), 'y','create_ssl_server_certs')) == 'y') {
589+
$inst->make_ispconfig_ssl_cert();
590+
}
591+
} else {
592+
swriteln('Certificate exists. Not creating a new one.');
593+
}
590594
}
591595

592596
if($conf['services']['web'] == true) {

install/update.php

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -519,6 +519,7 @@
519519
//** Configure ISPConfig
520520
swriteln('Updating ISPConfig');
521521

522+
$issue_asked = false;
522523
$issue_tried = false;
523524
// create acme vhost
524525
if($conf['nginx']['installed'] == true) {
@@ -544,15 +545,18 @@
544545
$inst->make_ispconfig_ssl_cert();
545546
$issue_tried = true;
546547
}
548+
$issue_asked = true;
547549
}
548550

549551
// Create SSL certs for non-webserver(s)?
550-
if(!file_exists('/usr/local/ispconfig/interface/ssl/ispserver.crt')) {
551-
if(!$issue_tried && strtolower($inst->simple_query('Do you want to create SSL certs for your server?', array('y', 'n'), 'y','create_ssl_server_certs')) == 'y') {
552-
$inst->make_ispconfig_ssl_cert();
553-
}
554-
} else {
555-
swriteln('Certificate exists. Not creating a new one.');
552+
if(!$issue_asked) {
553+
if(!file_exists('/usr/local/ispconfig/interface/ssl/ispserver.crt')) {
554+
if(!$issue_tried && strtolower($inst->simple_query('Do you want to create SSL certs for your server?', array('y', 'n'), 'y','create_ssl_server_certs')) == 'y') {
555+
$inst->make_ispconfig_ssl_cert();
556+
}
557+
} else {
558+
swriteln('Certificate exists. Not creating a new one.');
559+
}
556560
}
557561

558562
$inst->install_ispconfig();

0 commit comments

Comments
 (0)