From 8c663379dcb9859a060e07681cc9082c025cf203 Mon Sep 17 00:00:00 2001
From: polo <ordipolo@gmx.fr>
Date: Thu, 22 May 2025 16:59:25 +0200
Subject: classe Captcha

---
 src/controller/password.php | 62 ++++++++++++---------------------------------
 1 file changed, 16 insertions(+), 46 deletions(-)

(limited to 'src/controller/password.php')

diff --git a/src/controller/password.php b/src/controller/password.php
index 4748d9d..4b387ca 100644
--- a/src/controller/password.php
+++ b/src/controller/password.php
@@ -31,18 +31,18 @@ function createPassword(EntityManager $entityManager)
 {
 	// fonction exécutée à priori deux fois d'affilée: affichage puis traitement de la saisie
 
-	// II - traitement
 	unset($_SESSION['user']);
 
 	$captcha_solution = (isset($_SESSION['captcha']) && is_int($_SESSION['captcha'])) ? $_SESSION['captcha'] : 0;
-	$captcha = isset($_POST['captcha']) ? controlCaptchaInput($_POST['captcha']) : 0;
-
+	$captcha_try = isset($_POST['captcha']) ? Captcha::controlInput($_POST['captcha']) : 0;
+	
+	// II - traitement
 	$error = '';
-	if(!isset($_POST['captcha'])) // page rechargée
+	if(!isset($_POST['captcha'])) // page création d'un compte rechargée
 	{
 		//$error = '';
 	}
-	elseif($captcha == 0)
+	elseif($captcha_try == 0)
 	{
 		$error = 'error_non_valid_captcha';
 	}
@@ -50,7 +50,7 @@ function createPassword(EntityManager $entityManager)
 	{
 		//$error = '';
 	}
-	elseif($captcha != $captcha_solution) // le test!
+	elseif($captcha_try != $captcha_solution) // le test!
 	{
 		$error = 'bad_solution_captcha';
 	}
@@ -72,6 +72,8 @@ function createPassword(EntityManager $entityManager)
 		if(isset($password) && isset($login)
 			&& $password == $_POST['password'] && $login == $_POST['login'])
 		{
+			unset($_SESSION['captcha']);
+
 			// enregistrement et redirection
 			$password = password_hash($password, PASSWORD_DEFAULT);
 			$user = new App\Entity\User($login, $password);
@@ -90,9 +92,9 @@ function createPassword(EntityManager $entityManager)
 	}
 	
 	// inséré dans $captchaHtml puis dans $formulaireNouveauMDP
-	$captcha = createCaptcha();
+	$captcha = new Captcha;
 	// enregistrement de la réponse du captcha pour vérification
-	$_SESSION['captcha'] = $captcha[2]; // int
+	$_SESSION['captcha'] = $captcha->getSolution();
 
 
 	// I - affichage
@@ -129,14 +131,14 @@ function connect(LoginBuilder $builder, EntityManager $entityManager)
 	$_SESSION['admin'] = false;
 
 	$captcha_solution = (isset($_SESSION['captcha']) && is_int($_SESSION['captcha'])) ? $_SESSION['captcha'] : 0;
-	$captcha = isset($_POST['captcha']) ? controlCaptchaInput($_POST['captcha']) : 0;
+	$captcha_try = isset($_POST['captcha']) ? Captcha::controlInput($_POST['captcha']) : 0;
 
 	$error = '';
 	if(!isset($_POST['captcha'])) // page rechargée
 	{
 		//$error = '';
 	}
-	elseif($captcha == 0)
+	elseif($captcha_try == 0)
 	{
 		$error = 'error_non_valid_captcha';
 	}
@@ -144,7 +146,7 @@ function connect(LoginBuilder $builder, EntityManager $entityManager)
 	{
 		//$error = '';
 	}
-	elseif($captcha != $captcha_solution) // le test!
+	elseif($captcha_try != $captcha_solution) // le test!
 	{
 		$error = 'bad_solution_captcha';
 	}
@@ -162,8 +164,8 @@ function connect(LoginBuilder $builder, EntityManager $entityManager)
 		// enregistrement et redirection
 		if(!empty($user) && $login === $user->getLogin() && password_verify($password, $user->getPassword()))
 		{
-			session_start();
 			session_regenerate_id(true); // protection fixation de session, si l'attaquant a créé un cookie de session (attaque XSS), il est remplacé
+			//unset($_SESSION['captcha']);
 			$_SESSION['user'] = $login;
 			$_SESSION['admin'] = true;
 			$link = new URL(isset($_GET['from']) ? ['page' => $_GET['from']] : []);
@@ -178,9 +180,9 @@ function connect(LoginBuilder $builder, EntityManager $entityManager)
 	}
 
 	// inséré dans $captchaHtml puis dans $formulaireNouveauMDP
-	$captcha = createCaptcha();
+	$captcha = new Captcha;
 	// enregistrement de la réponse du captcha pour vérification
-	$_SESSION['captcha'] = $captcha[2]; // int
+	$_SESSION['captcha'] = $captcha->getSolution(); // int
 
 	// I - affichage
 	$title = "Connexion";
@@ -319,36 +321,4 @@ function disconnect()
 	isset($_GET['id']) ? $link->addParams(['id' => $_GET['id']]) : '';
 	header('Location: ' . $link);
 	die;
-}
-
-
-function createCaptcha(): array
-{
-	$a = rand(2, 9);
-	$b = rand(2, 9);
-	return array(toLettersFrench($a), toLettersFrench($b), $a * $b);
-}
-
-function toLettersFrench(int $number): string
-{
-	return match($number)
-	{
-		2 => 'deux',
-		3 => 'trois',
-		4 => 'quatre',
-		5 => 'cinq',
-		6 => 'six',
-		7 => 'sept',
-		8 => 'huit',
-		9 => 'neuf',
-		default => '', // erreur
-	};
-}
-
-// on veut des chiffres
-function controlCaptchaInput(string $captcha = '0'): int
-{
-    // $captcha est un POST donc une chaîne, '2.3' est acceptés
-    // (int) supprime les décimales
-    return (is_numeric($captcha) && $captcha == (int) $captcha) ? (int) $captcha : 0;
 }
\ No newline at end of file
-- 
cgit v1.2.3