From a1049a55a2c88e3fcd242ed13362f970792f8471 Mon Sep 17 00:00:00 2001
From: rudy <rmasson@cadoles.com>
Date: Tue, 24 Sep 2024 10:32:05 +0200
Subject: [PATCH] =?UTF-8?q?issue-39=20:=20close=20cursor=20pour=20requ?=
 =?UTF-8?q?=C3=AAte=20password?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/SQLLogin/SQLLoginConnect.php | 5 ++---
 src/Service/SQLLoginService.php  | 2 +-
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/SQLLogin/SQLLoginConnect.php b/src/SQLLogin/SQLLoginConnect.php
index 86b55a6..c2a7cfe 100644
--- a/src/SQLLogin/SQLLoginConnect.php
+++ b/src/SQLLogin/SQLLoginConnect.php
@@ -13,9 +13,8 @@ class SQLLoginConnect extends AbstractController
      * Méthode qui crée l'unique instance de la classe
      * si elle n'existe pas encore puis la retourne.
      *
-     * @return SQLLoginConnect
      */
-    public static function getInstance()
+    public static function getInstance(): SQLLoginConnect
     {
         if (is_null(self::$_instance)) {
             self::$_instance = new SQLLoginConnect();
@@ -24,7 +23,7 @@ class SQLLoginConnect extends AbstractController
         return self::$_instance;
     }
 
-    public function connect($urlDatabase, $dbUser, $dbPassword)
+    public function connect($urlDatabase, $dbUser, $dbPassword): PDO
     {
         return new PDO($urlDatabase, $dbUser, $dbPassword);
     }
diff --git a/src/Service/SQLLoginService.php b/src/Service/SQLLoginService.php
index ff43e78..46a23ab 100644
--- a/src/Service/SQLLoginService.php
+++ b/src/Service/SQLLoginService.php
@@ -75,6 +75,7 @@ class SQLLoginService extends AbstractController
             $query = $dbh->prepare($request);
             $query->execute([$this->sqlLoginRequest->getLoginColumnName() => $login]);
             $password = $query->fetch(PDO::FETCH_ASSOC);
+            $query->closeCursor();
         } catch (PDOException $e) {
             $this->loggerInterface->critical($e->getMessage());
             throw new LoginElementsConfigurationException($e->getMessage());
@@ -82,7 +83,6 @@ class SQLLoginService extends AbstractController
         if (!$password) {
             throw new Exception('Une erreur est survenue lors de la récupération des données');
         }
-
         return [
             $password[$this->sqlLoginRequest->getPasswordColumnName()],
             isset($password[$this->sqlLoginRequest->getSaltColumnName()]) ? $password[$this->sqlLoginRequest->getSaltColumnName()] : null,