Merge branch '10313' into 10400

CHANGES.md

@@ -24,8 +24,10 @@ 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.
+    - 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.
+
     - 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 = 'i' . $text;
+					$text = '_' . $text;
 				}
 				break;
 			case self::FILTER_INT:

core/module/user/user.php

@@ -365,6 +365,7 @@ 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);
@@ -373,29 +374,15 @@ class user extends common {
 				$this->getData(['config','connect','captcha'])
 				AND password_verify($this->getInput('userLoginCaptcha', helper::FILTER_INT), $this->getInput('userLoginCaptchaResult') ) === false )
 			{
-				//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
-				]);
+				$captcha = false;
 			} else {
+				$captcha = true;
+			}
 			/**
 			 * Aucun compte existant
 			 */
 			if ( !$this->getData(['user', $userId])) {
+				$logStatus = 'Compte inconnu';
 				//Stockage de l'IP
 				$this->setData([
 					'blacklist',
@@ -410,6 +397,7 @@ 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é',
@@ -419,7 +407,7 @@ class user extends common {
 				} else {
 					// Valeurs en sortie
 					$this->addOutput([
-							'notification' => 'Identifiant ou mot de passe incorrects'
+						'notification' => 'Captcha, identifiant ou mot de passe incorrects'
 					]);
 				}
 			/**
@@ -439,6 +427,7 @@ 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 ]);
@@ -462,6 +451,7 @@ 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']) ,
@@ -471,7 +461,8 @@ class user extends common {
 					}
 				// Sinon notification d'échec
 				} else {
-						$notification = 'Identifiant ou mot de passe incorrects';
+					$notification = 'Captcha, identifiant ou mot de passe incorrects';
+					$logStatus = $captcha === true ? 'Erreur de mot de passe' : 'Erreur de captcha';
 					// 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 ]);
@@ -484,18 +475,7 @@ 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
@@ -503,7 +483,19 @@ 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'];
 		}