CHANGES.md

@@ -24,10 +24,8 @@ Modifications :
         - Verrouillage des fichiers de données ouverts en écriture.
         - Message d'erreur littéral.
         - Sauvegarde des fichiers de données après un effacement et une écriture.
-        - Identifiant des noms de ressources (id de page , d'utilisateur, etc..) composés de nombres , remplacement du caractère de préfixe "i" par "_".
     - Google Analytics, option d'anonymisation.
-    - Procédure de connexion : les erreurs de captcha sont comptabilisées comme des échecs, allégement des messages d'information. Echecs de connexion, informations plus précises dans le journal de connexion.
-
+    - Procédure de connexion : les erreurs de captcha sont comptabilisées comme des échecs, allégement des messages d'information.
     - TinyMCE : ajout des scripts possibles.
 Correction :
     - Notification de commentaire, remplacement du nom de la page par le titre de l'article.

core/class/helper.class.php

@@ -273,7 +273,7 @@ class helper {
 				}
 				// Un ID ne peut pas être un entier, pour éviter les conflits avec le système de pagination
 				if(intval($text) !== 0) {
-					$text = '_' . $text;
+					$text = 'i' . $text;
 				}
 				break;
 			case self::FILTER_INT:

core/module/user/user.php

@@ -365,7 +365,6 @@ class user extends common {
 	 */
 	public function login() {
 		// Soumission du formulaire
-		$logStatus = '';
 		if($this->isPost()) {
 			// Lire Id du compte
 			$userId = $this->getInput('userLoginId', helper::FILTER_ID, true);
@@ -374,15 +373,29 @@ class user extends common {
 				$this->getData(['config','connect','captcha'])
 				AND password_verify($this->getInput('userLoginCaptcha', helper::FILTER_INT), $this->getInput('userLoginCaptchaResult') ) === false )
 			{
-				$captcha = false;
-			} else {
-				$captcha = true;
+				//self::$inputNotices['userLoginCaptcha'] = 'Incorrect';
+				$notification = 'Captcha incorrect !';
+				// Cas 1 le nombre de connexions est inférieur aux tentatives autorisées : incrément compteur d'échec
+				if ($this->getData(['user',$userId,'connectFail']) < $this->getData(['config', 'connect', 'attempt'])) {
+					$this->setData(['user',$userId,'connectFail',$this->getdata(['user',$userId,'connectFail']) + 1 ]);
 				}
+				// Cas 2 la limite du nombre de connexion est atteinte : placer le timer
+				if ( $this->getdata(['user',$userId,'connectFail']) == $this->getData(['config', 'connect', 'attempt'])	) {
+						$this->setData(['user',$userId,'connectTimeout', time()]);
+				}
+				// Cas 3 le délai de bloquage court
+				if ($this->getData(['user',$userId,'connectTimeout'])  + $this->getData(['config', 'connect', 'timeout']) > time() ) {
+					$notification = 'Accès bloqué ' . ($this->getData(['config', 'connect', 'timeout']) / 60) . ' minutes.';
+				}
+				// Valeurs en sortie
+				$this->addOutput([
+					'notification' => $notification
+				]);
+			} else {
 				/**
 				 * Aucun compte existant
 				 */
 				if ( !$this->getData(['user', $userId])) {
-				$logStatus = 'Compte inconnu';
 					//Stockage de l'IP
 					$this->setData([
 						'blacklist',
@@ -397,7 +410,6 @@ class user extends common {
 					$ipBlackList = helper::arrayCollumn($this->getData(['blacklist']), 'ip');
 					if ( $this->getData(['blacklist',$userId,'connectFail']) >= $this->getData(['config', 'connect', 'attempt'])
 						AND in_array($this->getData(['blacklist',$userId,'ip']),$ipBlackList) ) {
-					$logStatus = 'Compte inconnu verrouillé';
 						// Valeurs en sortie
 						$this->addOutput([
 							'notification' => 'Compte verrouillé',
@@ -407,7 +419,7 @@ class user extends common {
 					} else {
 						// Valeurs en sortie
 						$this->addOutput([
-						'notification' => 'Captcha, identifiant ou mot de passe incorrects'
+							'notification' => 'Identifiant ou mot de passe incorrects'
 						]);
 					}
 				/**
@@ -427,7 +439,6 @@ class user extends common {
 						AND $this->getData(['user',$userId,'connectFail']) < $this->getData(['config', 'connect', 'attempt'])
 						AND password_verify($this->getInput('userLoginPassword', helper::FILTER_STRING_SHORT, true), $this->getData(['user', $userId, 'password']))
 						AND $this->getData(['user', $userId, 'group']) >= self::GROUP_MEMBER
-					AND $captcha === true
 					) {
 						// RAZ
 						$this->setData(['user',$userId,'connectFail',0 ]);
@@ -451,7 +462,6 @@ class user extends common {
 								'state' => false
 							]);
 						} else {
-						$logStatus = 'Connexion réussie';
 							// Valeurs en sortie
 							$this->addOutput([
 								'notification' => 'Bienvenue ' . $this->getData(['user',$userId,'firstname']) . ' ' . $this->getData(['user',$userId,'lastname']) ,
@@ -461,8 +471,7 @@ class user extends common {
 						}
 					// Sinon notification d'échec
 					} else {
-					$notification = 'Captcha, identifiant ou mot de passe incorrects';
-					$logStatus = $captcha === true ? 'Erreur de mot de passe' : 'Erreur de captcha';
+						$notification = 'Identifiant ou mot de passe incorrects';
 						// Cas 1 le nombre de connexions est inférieur aux tentatives autorisées : incrément compteur d'échec
 						if ($this->getData(['user',$userId,'connectFail']) < $this->getData(['config', 'connect', 'attempt'])) {
 							$this->setData(['user',$userId,'connectFail',$this->getdata(['user',$userId,'connectFail']) + 1 ]);
@@ -475,7 +484,18 @@ class user extends common {
 						if ($this->getData(['user',$userId,'connectTimeout'])  + $this->getData(['config', 'connect', 'timeout']) > time() ) {
 							$notification = 'Accès bloqué ' . ($this->getData(['config', 'connect', 'timeout']) / 60) . ' minutes.';
 						}
-
+						// Journalisation
+						$dataLog = mb_detect_encoding(strftime('%d/%m/%y',time()), 'UTF-8', true)
+								? strftime('%d/%m/%y',time()) . ';' . strftime('%R',time()) . ';'
+								: utf8_encode(strftime('%d/%m/%y',time())) . ';' . utf8_encode(strftime('%R',time())) . ';' ;
+						$dataLog .= helper::getIp() . ';';
+						$dataLog .= $userId . ';' ;
+						$dataLog .= $this->getUrl() .';' ;
+						$dataLog .= 'échec de connexion' ;
+						$dataLog .= PHP_EOL;
+						if ($this->getData(['config','connect','log'])) {
+							file_put_contents(self::DATA_DIR . 'journal.log', $dataLog, FILE_APPEND);
+						}
 						// Valeurs en sortie
 						$this->addOutput([
 							'notification' => $notification
@@ -483,19 +503,7 @@ class user extends common {
 					}
 				}
 			}
-		// Journalisation
-		$dataLog = mb_detect_encoding(strftime('%d/%m/%y',time()), 'UTF-8', true)
-				? strftime('%d/%m/%y',time()) . ';' . strftime('%R',time()) . ';'
-				: utf8_encode(strftime('%d/%m/%y',time())) . ';' . utf8_encode(strftime('%R',time())) . ';' ;
-		$dataLog .= helper::getIp() . ';';
-		$dataLog .= $this->getInput('userLoginId', helper::FILTER_ID) . ';' ;
-		$dataLog .= $this->getUrl() .';' ;
-		$dataLog .= $logStatus ;
-		$dataLog .= PHP_EOL;
-		if ($this->getData(['config','connect','log'])) {
-			file_put_contents(self::DATA_DIR . 'journal.log', $dataLog, FILE_APPEND);
 		}
-		// Stockage des cookies
 		if (!empty($_COOKIE['ZWII_USER_ID'])) {
 			self::$userId = $_COOKIE['ZWII_USER_ID'];
 		}