10.2.dev20 Liste noire

core/core.php

@@ -39,7 +39,7 @@ class common {
 	const ACCESS_TIMER = 1800;
 
 	// Numéro de version
-	const ZWII_VERSION = '10.2.00.dev19';
+	const ZWII_VERSION = '10.2.00.dev20';
 	const ZWII_UPDATE_CHANNEL = "v10";
 
 	public static $actions = [];

core/module/config/config.php

@@ -656,7 +656,7 @@ class config extends common {
 		$d = $this->getData(['blacklist']);
 		$data = '';
 		foreach ($d as $key => $item) {
-			$data .=  $key . ';' . $item['ip'] . PHP_EOL;
+			$data .= strftime('%d/%m/%y',$item['time']) . ';' . strftime('%R',$item['time']) . ';' . $key . ';' . $item['ip']  . PHP_EOL;
 		}
 		$fileName = self::TEMP_DIR . 'blacklist.log';
 		file_put_contents($fileName,$data);
@@ -668,7 +668,7 @@ class config extends common {
 		$this->addOutput([
 			'display' => self::DISPLAY_RAW
 		]);
-		unlink(self::TEMP_DIR . 'blacklist.tmp');
+		unlink(self::TEMP_DIR . 'blacklist.log');
 		// Valeurs en sortie
 		$this->addOutput([
 			'title' => 'Configuration',

core/module/config/view/index/index.php

@@ -350,24 +350,30 @@
 <div class="row">
 	<div class="col12">
 		<div class="block">
-			<h4>Sécurité</h4>
+			<h4>Sécurité de la connexion</h4>
 			<div class="row">
 				<div class="col3">
 					<?php echo template::select('configConnectAttempt', $module::$connectAttempt , [
-						'label' => 'Tentatives de logins',
+						'label' => 'Connexions successives',
 						'selected' => $this->getData(['config', 'connect', 'attempt'])
 					]); ?>
 				</div>
 				<div class="col3">
 					<?php echo template::select('configConnectTimeout', $module::$connectTimeout , [
-						'label' => 'Durée du blocage',
+						'label' => 'Blocage après échecs',
 						'selected' => $this->getData(['config', 'connect', 'timeout'])
 					]); ?>
 				</div>
 				<div class="col3 verticalAlignBottom">
+					<label id="helpBlacklist">Blocage des comptes inconnus :
+						<?php echo template::help(
+						'La liste noire énumère les tentatives de connexion à partir de comptes inexistants. Sont stockés : la date, l\'heure, le nom du compte et l\'IP.
+						Après le nombre de tentatives autorisées, l\'IP et le compte sont bloquées.');
+						?>
+					</label>
 					<?php echo template::button('configConnectblacListDownload', [
 						'href' => helper::baseUrl() . 'config/blacklistDownload',
-						'value' => 'IP liste noire',
+						'value' => 'Télécharger liste noire',
 						'ico' => 'download'
 					]); ?>
 				</div>
@@ -375,11 +381,18 @@
 					<?php echo template::button('ConfigConnectReset', [
 						'class' => 'buttonRed',
 						'href' => helper::baseUrl() . 'config/blacklistReset',
-						'value' => 'Réinitialisation liste',
+						'value' => 'Réinitialiser liste',
 						'ico' => 'cancel'
 					]); ?>
 				</div>
 			</div>
+		</div>
+	</div>
+</div>
+<div class="row">
+	<div class="col12">
+		<div class="block">
+			<h4>Journalisation</h4>
 			<div class="row">
 				<div class="col3 verticalAlignBottom">
 					<?php echo template::checkbox('configConnectLog', true, 'Activer la journalisation', [
@@ -389,7 +402,7 @@
 				<div class="col3 offset3">
 					<?php echo template::button('ConfigLogDownload', [
 						'href' => helper::baseUrl() . 'config/logDownload',
-						'value' => 'Téléchargement du journal',
+						'value' => 'Télécharger journal',
 						'ico' => 'download'
 					]); ?>
 				</div>
@@ -397,7 +410,7 @@
 					<?php echo template::button('ConfigLogReset', [
 						'class' => 'buttonRed',
 						'href' => helper::baseUrl() . 'config/logReset',
-						'value' => 'Réinitialisation du journal',
+						'value' => 'Réinitialiser journal',
 						'ico' => 'cancel'
 					]); ?>
 				</div>
@@ -418,7 +431,7 @@
 						'value' => $this->getData(['config', 'analyticsId'])
 					]); ?>
 				</div>
-				<div class="col3 verticalAlignBottom">
+				<div class="col3 offset3 verticalAlignBottom">
 					<?php echo template::button('configScriptHead', [
 						'href' => helper::baseUrl() . 'config/script/head',
 						'value' => 'Script dans head',

core/module/user/user.php

@@ -332,20 +332,28 @@ class user extends common {
 		// Soumission du formulaire
 		if($this->isPost()) {
 			$userId = $this->getInput('userLoginId', helper::FILTER_ID, true);
-			// le userId n'existe pas, créer une entré dans la liste noire
+
+			// le userId n'existe pas, créer une entrée dans la liste noire
 			if( !$this->getData(['user', $userId])) {
 				//Stockage de l'IP
 				$this->setData([
 					'blacklist',
 					$userId,
 					[
+						'time' => time(),
 						'ip' => $_SERVER['REMOTE_ADDR'],
 						'connectFail' => $this->getData(['blacklist',$userId,'connectFail']) ? $this->getData(['blacklist',$userId,'connectFail']) + 1 : 1
 					]
 				]);
+				$notification = 'Identifiant ou mot de passe incorrect';
 			}
-			// Contrôle du timeout pas de vérification du mot de passe si le temps est dépassé.
+
-			// Connexion si les informations sont correctes
+			/**
+			 *  Compte valide : 
+			 *  Timeout entre nombre de tentatives autorisées
+			 *  Mot de passe
+			 *  Groupe
+			*/
 			if( $this->getData(['user',$userId,'connectTimeout']) + $this->getData(['config', 'connect', 'timeout']) < time() &&
 				$this->getData(['user',$userId,'connectFail']) < $this->getData(['config', 'connect', 'attempt']) &&
 				password_verify($this->getInput('userLoginPassword', helper::FILTER_STRING_SHORT, true), $this->getData(['user', $userId, 'password']))
@@ -381,10 +389,12 @@ class user extends common {
 			// Sinon notification d'échec
 			else {
 				// L'utilisateur existe : incrémenter le compteur d'échec de connexion
-				if ( is_array($this->getdata(['user',$userId])) ) {
+				if ( is_array($this->getdata(['user',$userId]))
+				) {
 					$this->setData(['user',$userId,'connectFail',$this->getdata(['user',$userId,'connectFail']) + 1 ]);
 					// Mettre à jour le timer et notifier
-					if ( $this->getdata(['user',$userId,'connectFail']) > $this->getData(['config', 'connect', 'attempt'])) {
+					if ( $this->getdata(['user',$userId,'connectFail']) > $this->getData(['config', 'connect', 'attempt'])
+					) {
 						$notification = 'Trop de tentatives, accès bloqué durant ' . ($this->getData(['config', 'connect', 'timeout']) / 60) . ' minutes.';
 						// Ne pas incrémenter le timer si actif
 						if ($this->getData(['user',$userId,'connectTimeout'])  + $this->getData(['config', 'connect', 'timeout']) < time() ) {
@@ -393,15 +403,14 @@ class user extends common {
 					} else {
 						$notification = 'Identifiant ou mot de passe incorrect';
 					}
-					// L'utilisateur n'existe pas
+				// L'utilisateur n'existe pas
-					// Bloquer l'IP après les tentatives autorisées,
+				// Bloquer l'IP après les tentatives autorisées avec ce compte,
 				} elseif (
 							$this->getData(['blacklist',$userId,'connectFail']) > $this->getData(['config', 'connect', 'attempt']) ||
 							array_search($_SERVER['REMOTE_ADDR'],helper::arrayCollumn($this->getData(['blacklist']), 'ip'))
 						 ) {
-							$notification = 'Trop de tentatives, accès bloqué durant ' . ($this->getData(['config', 'connect', 'timeout']) / 60) . ' minutes.';
+							$notification = 'Trop de tentatives, compte verrouillé';
 				}
-
 				// Journalisation
 				$dataLog = strftime('%d/%m/%y',time()) . ';' . strftime('%R',time()) . ';' ;
 				$dataLog .= $_SERVER['REMOTE_ADDR'] . ';' ;

core/module/user/view/login/login.php

@@ -13,7 +13,9 @@
 	</div>
 	<div class="row">
 		<div class="col6">
-			<?php echo template::checkbox('userLoginLongTime', true, 'Se souvenir de moi'); ?>
+			<?php echo template::checkbox('userLoginLongTime', true, 'Se souvenir de moi', [
+			'help' => 'La session est close à la fermeture du navigateur.'
+			]); ?>
 
 		</div>
 		<div class="col6 textAlignRight">