From aef2832197d4627cf87fdd5ebf69b25c68a411a7 Mon Sep 17 00:00:00 2001 From: Marius Burkard <m.burkard@pixcept.de> Date: Wed, 17 Feb 2016 05:03:50 -0500 Subject: [PATCH] Merge branch 'stable-3.1' --- interface/web/login/lib/lang/en.lng | 8 ++ interface/web/login/password_reset.php | 62 +++++++++++++++++++- install/dist/lib/gentoo.lib.php | 3 - install/update.php | 39 ++++++------ server/plugins-available/software_update_plugin.inc.php | 1 server/plugins-available/mysql_clientdb_plugin.inc.php | 7 -- install/install.php | 2 install/lib/installer_base.lib.php | 15 ---- install/sql/ispconfig3.sql | 2 install/sql/incremental/upd_dev_collection.sql | 4 + server/plugins-available/apache2_plugin.inc.php | 6 ++ interface/web/login/lib/lang/de.lng | 8 ++ server/plugins-available/nginx_plugin.inc.php | 10 ++ 13 files changed, 115 insertions(+), 52 deletions(-) diff --git a/install/dist/lib/gentoo.lib.php b/install/dist/lib/gentoo.lib.php index 4080bed..b457474 100644 --- a/install/dist/lib/gentoo.lib.php +++ b/install/dist/lib/gentoo.lib.php @@ -472,9 +472,6 @@ $this->error('Unable to create user for powerdns database Error: '.$this->db->errorMessage); } - //* Reload database privelages - $this->db->query('FLUSH PRIVILEGES;'); - //* load the powerdns databse dump if($conf['mysql']['admin_password'] == '') { caselog("mysql --default-character-set=".$conf['mysql']['charset']." -h '".$conf['mysql']['host']."' -u '".$conf['mysql']['admin_user']."' '".$conf['powerdns']['database']."' < '".ISPC_INSTALL_ROOT."/install/sql/powerdns.sql' &> /dev/null", diff --git a/install/install.php b/install/install.php index 1c413f9..e07546e 100644 --- a/install/install.php +++ b/install/install.php @@ -208,7 +208,7 @@ } //* Initialize the MySQL server connection - if(@mysqli_connect($tmp_mysql_server_host, $tmp_mysql_server_admin_user, $tmp_mysql_server_admin_password, $tmp_mysql_server_database, (int)$tmp_mysql_server_port)) { + if(@mysqli_connect($tmp_mysql_server_host, $tmp_mysql_server_admin_user, $tmp_mysql_server_admin_password, '', (int)$tmp_mysql_server_port)) { $conf['mysql']['host'] = $tmp_mysql_server_host; $conf['mysql']['port'] = $tmp_mysql_server_port; $conf['mysql']['admin_user'] = $tmp_mysql_server_admin_user; diff --git a/install/lib/installer_base.lib.php b/install/lib/installer_base.lib.php index f2b7041..363a38c 100644 --- a/install/lib/installer_base.lib.php +++ b/install/lib/installer_base.lib.php @@ -263,18 +263,14 @@ } // Delete ISPConfig user in the local database, in case that it exists - $this->db->query("DELETE FROM mysql.user WHERE User = ? AND Host = ?", $conf['mysql']['ispconfig_user'], $from_host); - $this->db->query("DELETE FROM mysql.db WHERE Db = ? AND Host = ?", $conf['mysql']['database'], $from_host); - $this->db->query('FLUSH PRIVILEGES'); + $this->db->query("DROP USER ?@?", $conf['mysql']['ispconfig_user'], $from_host); + $this->db->query("DROP DATABASE IF EXISTS ?", $conf['mysql']['database']); //* Create the ISPConfig database user in the local database $query = 'GRANT SELECT, INSERT, UPDATE, DELETE ON ?? TO ?@? IDENTIFIED BY ?'; if(!$this->db->query($query, $conf['mysql']['database'] . ".*", $conf['mysql']['ispconfig_user'], $from_host, $conf['mysql']['ispconfig_password'])) { $this->error('Unable to create database user: '.$conf['mysql']['ispconfig_user'].' Error: '.$this->db->errorMessage); } - - //* Reload database privelages - $this->db->query('FLUSH PRIVILEGES;'); //* Set the database name in the DB library $this->db->setDBName($conf['mysql']['database']); @@ -655,10 +651,6 @@ } } - /* - * It is all done. Relod the rights... - */ - $this->dbmaster->query('FLUSH PRIVILEGES'); } } @@ -1471,9 +1463,6 @@ if(!$this->db->query($query, $conf['powerdns']['database'] . '.*', $conf['mysql']['ispconfig_user'])) { $this->error('Unable to create user for powerdns database Error: '.$this->db->errorMessage); } - - //* Reload database privelages - $this->db->query('FLUSH PRIVILEGES'); //* load the powerdns databse dump if($conf['mysql']['admin_password'] == '') { diff --git a/install/sql/incremental/upd_dev_collection.sql b/install/sql/incremental/upd_dev_collection.sql index f712ef1..56cef0c 100644 --- a/install/sql/incremental/upd_dev_collection.sql +++ b/install/sql/incremental/upd_dev_collection.sql @@ -216,3 +216,7 @@ ALTER TABLE `client_template` ADD COLUMN `limit_ssl_letsencrypt` enum('n','y') NOT NULL DEFAULT 'n' AFTER `limit_ssl`; ALTER TABLE `client` ADD COLUMN `limit_directive_snippets` ENUM( 'n', 'y' ) NOT NULL DEFAULT 'n' AFTER `limit_backup`; ALTER TABLE `client_template` ADD COLUMN `limit_directive_snippets` ENUM( 'n', 'y' ) NOT NULL DEFAULT 'n' AFTER `limit_backup`; + +ALTER TABLE `sys_user` + ADD COLUMN `lost_password_hash` VARCHAR(50) NOT NULL DEFAULT '', + ADD COLUMN `lost_password_reqtime` DATETIME NULL default NULL; diff --git a/install/sql/ispconfig3.sql b/install/sql/ispconfig3.sql index bf7975c..53d2753 100644 --- a/install/sql/ispconfig3.sql +++ b/install/sql/ispconfig3.sql @@ -1791,6 +1791,8 @@ `id_rsa` VARCHAR( 2000 ) NOT NULL default '', `ssh_rsa` VARCHAR( 600 ) NOT NULL default '', `lost_password_function` tinyint(1) NOT NULL default '1', + `lost_password_hash` VARCHAR(50) NOT NULL default '', + `lost_password_reqtime` DATETIME NULL default NULL, PRIMARY KEY (`userid`) ) ENGINE=MyISAM DEFAULT CHARSET=utf8 AUTO_INCREMENT=1 ; diff --git a/install/update.php b/install/update.php index 9b5b9ba..8f818df 100644 --- a/install/update.php +++ b/install/update.php @@ -321,33 +321,34 @@ if($conf['services']['mail']) { //** Configure postfix - if($inst->reconfigure_app('Postfix', $reconfigure_services_answer)) { + if($inst->reconfigure_app('Postfix and IMAP/POP3', $reconfigure_services_answer)) { swriteln('Configuring Postfix'); $inst->configure_postfix('dont-create-certs'); + + if($conf['dovecot']['installed'] == true) { + //* Configure dovecot + swriteln('Configuring Dovecot'); + $inst->configure_dovecot(); + } elseif ($conf['courier']['installed'] == true) { + //** Configure saslauthd + swriteln('Configuring SASL'); + $inst->configure_saslauthd(); + + //** Configure PAM + swriteln('Configuring PAM'); + $inst->configure_pam(); + + //* Configure courier + swriteln('Configuring Courier'); + $inst->configure_courier(); + } + } //** Configure mailman if($conf['mailman']['installed'] == true && $inst->reconfigure_app('Mailman', $reconfigure_services_answer)) { swriteln('Configuring Mailman'); $inst->configure_mailman('update'); - } - - if($conf['dovecot']['installed'] == true && $inst->reconfigure_app('Dovecot', $reconfigure_services_answer)) { - //* Configure dovecot - swriteln('Configuring Dovecot'); - $inst->configure_dovecot(); - } elseif ($conf['courier']['installed'] == true && $inst->reconfigure_app('Courier', $reconfigure_services_answer)) { - //** Configure saslauthd - swriteln('Configuring SASL'); - $inst->configure_saslauthd(); - - //** Configure PAM - swriteln('Configuring PAM'); - $inst->configure_pam(); - - //* Configure courier - swriteln('Configuring Courier'); - $inst->configure_courier(); } //** Configure Spamasassin diff --git a/interface/web/login/lib/lang/de.lng b/interface/web/login/lib/lang/de.lng index 9a41ad8..445291c 100644 --- a/interface/web/login/lib/lang/de.lng +++ b/interface/web/login/lib/lang/de.lng @@ -5,10 +5,13 @@ $wb['error_user_too_many_logins'] = 'Zu viele falsche Logins. Bitte warten Sie 15 Minuten.'; $wb['pass_reset_txt'] = 'Ein neues Passwort wird generiert und an die E-Mail Adresse, die in ihren Benutzereinstellungen hinterlegt ist, gesendet, wenn Ihre eingegeben E-Mail-Adresse mit der hinterlegten E-Mail-Adresse übereinstimmt.'; $wb['pw_reset'] = 'Das Passwort wurde zurückgesetzt und wird per E-Mail an Sie geschickt.'; +$wb['pw_reset_act'] = 'Ein Aktivierungslink wurde per E-Mail an Sie geschickt. Bitte bestätigen Sie die Anforderung eines neuen Passwortes.'; $wb['pw_error'] = 'Benutzername oder E-Mail Adresse stimmen nicht überein.'; $wb['pw_error_noinput'] = 'Bitte geben Sie Ihre E-Mail Adresse und Ihren Benutzernamen ein.'; $wb['pw_reset_mail_msg'] = 'Das Passwort wurde zurückgesetzt. Das neue Passwort lautet: '; -$wb['pw_reset_mail_title'] = 'Ihr Passwort wurde zurückgesetzt.'; +$wb['pw_reset_mail_title'] = 'Ihr Passwort wurde zurückgesetzt'; +$wb['pw_reset_act_mail_msg'] = 'Bitte bestätigen Sie die Anforderung eines neuen Passworts, indem Sie folgenden Link besuchen: '; +$wb['pw_reset_act_mail_title'] = 'Anforderung zum Zurücksetzen des Passworts'; $wb['user_regex_error'] = 'Benutzername beinhaltet nicht erlaubte Zeichen oder ist länger als 64 Zeichen.'; $wb['pw_error_length'] = 'Die Passwortlänge ist < 1 oder > 64 Zeichen.'; $wb['login_txt'] = 'Anmelden'; @@ -25,4 +28,7 @@ $wb['stay_logged_in_txt'] = 'Dauerhaft eingeloggt bleiben'; $wb['email_error'] = 'Email contains unallowed characters or has a invalid format.'; $wb['lost_password_function_disabled_txt'] = 'Die Passwort vergessen Funktion steht für diesen Benutzer nicht zur Verfügung.'; +$wb['lost_password_function_wait_txt'] = 'Sie können im Moment kein neues Passwort anfordern. Bitte warten Sie einige Minuten.'; +$wb['lost_password_function_expired_txt'] = 'Der Passwortlink ist abgelaufen. Bitte fordern Sie einen neuen an.'; +$wb['lost_password_function_denied_txt'] = 'Dieser Passwortlink ist ungültig.'; ?> diff --git a/interface/web/login/lib/lang/en.lng b/interface/web/login/lib/lang/en.lng index bf2edfe..2f9fcdb 100644 --- a/interface/web/login/lib/lang/en.lng +++ b/interface/web/login/lib/lang/en.lng @@ -5,10 +5,13 @@ $wb['error_user_too_many_logins'] = "To many wrong login's, Please retry it after 15 minutes"; $wb['pass_reset_txt'] = 'A new password will be generated and send to your email address if the email address entered below matches the email address in your client settings.'; $wb['pw_reset'] = 'The password has been reset and send to your email address.'; +$wb['pw_reset_act'] = 'You have been sent an activation link. Please visit the link to confirm your password request.'; $wb['pw_error'] = 'Username or email address does not match.'; $wb['pw_error_noinput'] = 'Please enter email address and username.'; $wb['pw_reset_mail_msg'] = 'The password to your ISPConfig 3 control panel account has been reset. The new password is: '; -$wb['pw_reset_mail_title'] = 'ISPConfig 3 Control panel password has been reset.'; +$wb['pw_reset_mail_title'] = 'ISPConfig 3 Control panel password has been reset'; +$wb['pw_reset_act_mail_title'] = 'Confirm ISPConfig 3 Control panel password reset'; +$wb['pw_reset_act_mail_msg'] = 'Please confirm that your want to reset your ISPConfig 3 control panel account password by visiting the following activation link: '; $wb['user_regex_error'] = 'Username contains unallowed characters or is longer than 64 characters.'; $wb['pw_error_length'] = 'The password length is < 1 or > 64 characters.'; $wb['email_error'] = 'Email contains unallowed characters or has a invalid format.'; @@ -25,4 +28,7 @@ $wb['theme_not_compatible'] = 'The chosen theme is not compatible with the current ISPConfig version. Please check for a new version of the theme.<br />The default theme as been activated automatically.'; $wb['stay_logged_in_txt'] = 'Keep me logged in'; $wb['lost_password_function_disabled_txt'] = 'The lost password function is not available for this user.'; +$wb['lost_password_function_wait_txt'] = 'You cannot request a new password, yet. Please wait a few minutes.'; +$wb['lost_password_function_expired_txt'] = 'This activation link has expired. Please request a new one.'; +$wb['lost_password_function_denied_txt'] = 'This activation link is not valid.'; ?> diff --git a/interface/web/login/password_reset.php b/interface/web/login/password_reset.php index a92ff85..dbb545d 100644 --- a/interface/web/login/password_reset.php +++ b/interface/web/login/password_reset.php @@ -48,12 +48,11 @@ $continue = true; if(isset($_POST['username']) && $_POST['username'] != '' && $_POST['email'] != '' && $_POST['username'] != 'admin') { - if(!preg_match("/^[\w\.\-\_]{1,64}$/", $_POST['username'])) { $app->tpl->setVar("error", $wb['user_regex_error']); $continue = false; } - if(!preg_match("/^\w+[\w.-]*\w+@\w+[\w.-]*\w+\.[a-z]{2,10}$/i", $_POST['email'])) { + if(!filter_var($_POST['email'], FILTER_VALIDATE_EMAIL)) { $app->tpl->setVar("error", $wb['email_error']); $continue = false; } @@ -61,10 +60,65 @@ $username = $_POST['username']; $email = $_POST['email']; - $client = $app->db->queryOneRecord("SELECT client.*, sys_user.lost_password_function FROM client,sys_user WHERE client.username = ? AND client.email = ? AND client.client_id = sys_user.client_id", $username, $email); + $client = $app->db->queryOneRecord("SELECT client.*, sys_user.lost_password_function, sys_user.lost_password_hash, IF(sys_user.lost_password_reqtime IS NOT NULL AND DATE_SUB(NOW(), INTERVAL 15 MINUTE) < sys_user.lost_password_reqtime, 1, 0) as `lost_password_wait` FROM client,sys_user WHERE client.username = ? AND client.email = ? AND client.client_id = sys_user.client_id", $username, $email); if($client['lost_password_function'] == 0) { $app->tpl->setVar("error", $wb['lost_password_function_disabled_txt']); + } elseif($client['lost_password_wait'] == 1) { + $app->tpl->setVar("error", $wb['lost_password_function_wait_txt']); + } elseif ($continue) { + if($client['client_id'] > 0) { + $username = $client['username']; + $password_hash = sha1(uniqid('ispc_pw')); + $app->db->query("UPDATE sys_user SET lost_password_reqtime = NOW(), lost_password_hash = ? WHERE username = ?", $password_hash, $username); + $app->tpl->setVar("message", $wb['pw_reset_act']); + + $server_domain = (isset($_SERVER['SERVER_NAME']) ? $_SERVER['SERVER_NAME'] : $_SERVER['HTTP_HOST']); + if($server_domain == '_') { + $tmp = explode(':',$_SERVER["HTTP_HOST"]); + $server_domain = $tmp[0]; + unset($tmp); + } + if(!isset($_SERVER['HTTPS']) || $_SERVER['HTTPS'] != 'on') $server_domain = 'http://' . $server_domain; + else $server_domain = 'https://' . $server_domain; + + if(isset($_SERVER['SERVER_PORT']) && $_SERVER['SERVER_PORT'] != '443') $server_domain .= ':' . $_SERVER['SERVER_PORT']; + + $app->uses('getconf,ispcmail'); + $mail_config = $server_config_array['mail']; + if($mail_config['smtp_enabled'] == 'y') { + $mail_config['use_smtp'] = true; + $app->ispcmail->setOptions($mail_config); + } + $app->ispcmail->setSender($mail_config['admin_mail'], $mail_config['admin_name']); + $app->ispcmail->setSubject($wb['pw_reset_act_mail_title']); + $app->ispcmail->setMailText($wb['pw_reset_act_mail_msg'].$server_domain . '/login/password_reset.php?username=' . urlencode($username) . '&hash=' . urlencode($password_hash)); + $app->ispcmail->send(array($client['contact_name'] => $client['email'])); + $app->ispcmail->finish(); + + $app->tpl->setVar("msg", $wb['pw_reset_act']); + } else { + $app->tpl->setVar("error", $wb['pw_error']); + } + } +} elseif(isset($_GET['username']) && $_GET['username'] != '' && $_GET['hash'] != '') { + + if(!preg_match("/^[\w\.\-\_]{1,64}$/", $_GET['username'])) { + $app->tpl->setVar("error", $wb['user_regex_error']); + $continue = false; + } + + $username = $_GET['username']; + $hash = $_GET['hash']; + + $client = $app->db->queryOneRecord("SELECT client.*, sys_user.lost_password_function, sys_user.lost_password_hash, IF(sys_user.lost_password_reqtime IS NULL OR DATE_SUB(NOW(), INTERVAL 1 DAY) > sys_user.lost_password_reqtime, 1, 0) as `lost_password_expired` FROM client,sys_user WHERE client.username = ? AND client.client_id = sys_user.client_id", $username); + + if($client['lost_password_function'] == 0) { + $app->tpl->setVar("error", $wb['lost_password_function_disabled_txt']); + } elseif($client['lost_password_expired'] == 1) { + $app->tpl->setVar("error", $wb['lost_password_function_expired_txt']); + } elseif($client['lost_password_hash'] != $hash) { + $app->tpl->setVar("error", $wb['lost_password_function_denied_txt']); } elseif ($continue) { if($client['client_id'] > 0) { $server_config_array = $app->getconf->get_global_config(); @@ -75,7 +129,7 @@ $new_password_encrypted = $app->auth->crypt_password($new_password); $username = $client['username']; - $app->db->query("UPDATE sys_user SET passwort = ? WHERE username = ?", $new_password_encrypted, $username); + $app->db->query("UPDATE sys_user SET passwort = ?, lost_password_hash = '', lost_password_reqtime = NULL WHERE username = ?", $new_password_encrypted, $username); $app->db->query("UPDATE client SET password = ? WHERE username = ?", $new_password_encrypted, $username); $app->tpl->setVar("message", $wb['pw_reset']); diff --git a/server/plugins-available/apache2_plugin.inc.php b/server/plugins-available/apache2_plugin.inc.php index 52a8687..3237bb5 100644 --- a/server/plugins-available/apache2_plugin.inc.php +++ b/server/plugins-available/apache2_plugin.inc.php @@ -1052,6 +1052,10 @@ } } } + + foreach($sub_prefixes as $s) { + $temp_domains[] = $s . $aliasdomain['domain']; + } } } } @@ -1137,6 +1141,7 @@ $lddomain = $domain; $subdomains = null; $aliasdomains = null; + $sub_prefixes = array(); //* be sure to have good domain if($data['new']['subdomain'] == "www" OR $data['new']['subdomain'] == "*") { @@ -1148,6 +1153,7 @@ if(is_array($subdomains)) { foreach($subdomains as $subdomain) { $temp_domains[] = $subdomain['domain']; + $sub_prefixes[] = str_replace($domain, "", $subdomain['domain']); } } diff --git a/server/plugins-available/mysql_clientdb_plugin.inc.php b/server/plugins-available/mysql_clientdb_plugin.inc.php index 7a5128a..743d27a 100644 --- a/server/plugins-available/mysql_clientdb_plugin.inc.php +++ b/server/plugins-available/mysql_clientdb_plugin.inc.php @@ -228,7 +228,6 @@ } - $link->query('FLUSH PRIVILEGES;'); $link->close(); } } @@ -433,7 +432,6 @@ } } // Database is not active, so stop processing here - $link->query('FLUSH PRIVILEGES;'); $link->close(); return; } @@ -570,8 +568,6 @@ } } - - $link->query('FLUSH PRIVILEGES;'); $link->close(); } @@ -621,7 +617,6 @@ $app->log('Error while dropping MySQL database: '.$data['old']['database_name'].' '.$link->error, LOGLEVEL_WARNING); } - $link->query('FLUSH PRIVILEGES;'); $link->close(); } @@ -685,7 +680,6 @@ } } - $link->query('FLUSH PRIVILEGES;'); $link->close(); } @@ -721,7 +715,6 @@ } } - $link->query('FLUSH PRIVILEGES;'); $link->close(); } diff --git a/server/plugins-available/nginx_plugin.inc.php b/server/plugins-available/nginx_plugin.inc.php index 33d5537..b50d4eb 100644 --- a/server/plugins-available/nginx_plugin.inc.php +++ b/server/plugins-available/nginx_plugin.inc.php @@ -1253,9 +1253,10 @@ // default values $temp_domains = array(); - $lddomain = $domain; - $subdomains = null; + $lddomain = $domain; + $subdomains = null; $aliasdomains = null; + $sub_prefixes = array(); //* be sure to have good domain if($data['new']['subdomain'] == "www" OR $data['new']['subdomain'] == "*") { @@ -1267,6 +1268,7 @@ if(is_array($subdomains)) { foreach($subdomains as $subdomain) { $temp_domains[] = $subdomain['domain']; + $sub_prefixes[] = str_replace($domain, "", $subdomain['domain']); } } @@ -1278,6 +1280,10 @@ if(isset($aliasdomain['subdomain']) && ! empty($aliasdomain['subdomain'])) { $temp_domains[] = $aliasdomain['subdomain'] . "." . $aliasdomain['domain']; } + + foreach($sub_prefixes as $s) { + $temp_domains[] = $s . $aliasdomain['domain']; + } } } diff --git a/server/plugins-available/software_update_plugin.inc.php b/server/plugins-available/software_update_plugin.inc.php index 83fb831..640d7a9 100644 --- a/server/plugins-available/software_update_plugin.inc.php +++ b/server/plugins-available/software_update_plugin.inc.php @@ -267,7 +267,6 @@ $app->log('Unable to create database user'.$db_config['database_user'].' '.mysqli_error($link), LOGLEVEL_ERROR); } - mysqli_query($link, "FLUSH PRIVILEGES;"); mysqli_close($link); } -- Gitblit v1.9.1