From 5aacd981b4a30bb81f470a39379835fe67811a05 Mon Sep 17 00:00:00 2001
From: rudy <rmasson@cadoles.com>
Date: Mon, 19 Jun 2023 14:22:05 +0200
Subject: [PATCH] issue-19: ajout de logs des exceptions PDOExceptions

---
 src/Service/SQLLoginService.php | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/Service/SQLLoginService.php b/src/Service/SQLLoginService.php
index 16b3d45..2e1d549 100644
--- a/src/Service/SQLLoginService.php
+++ b/src/Service/SQLLoginService.php
@@ -6,18 +6,20 @@ use App\SQLLogin\SQLLoginConnect;
 use App\SQLLogin\SQLLoginRequest;
 use PDO;
 use PDOException;
+use Psr\Log\LoggerInterface;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 
 class SQLLoginService extends AbstractController
 {
     public SQLLoginRequest $sqlLoginRequest;
 
-    public function __construct(SQLLoginRequest $sqlLoginRequest)
+    public function __construct(SQLLoginRequest $sqlLoginRequest, private LoggerInterface $loggerInterface)
     {
         $this->sqlLoginRequest = $sqlLoginRequest;
+        $this->loggerInterface = $loggerInterface;
     }
 
-    public function fetchDatas(string $login)
+    public function fetchDatas(string $login): array
     {
         try {
             $dbh = $this->getConnection();
@@ -29,7 +31,7 @@ class SQLLoginService extends AbstractController
             $datas = $query->fetch(PDO::FETCH_ASSOC);
         } catch (PDOException $e) {
             \Sentry\captureException($e);
-
+            $this->loggerInterface->critical($e->getMessage());
             throw new PDOException();
         }
 
@@ -45,7 +47,7 @@ class SQLLoginService extends AbstractController
             $query->execute([$this->sqlLoginRequest->getLoginColumnName() => $login]);
             $password = $query->fetch(PDO::FETCH_ASSOC);
         } catch (PDOException $e) {
-            \Sentry\captureException($e);
+            $this->loggerInterface->critical($e->getMessage());
             throw new PDOException();
         }
         if ($password) {
@@ -58,7 +60,7 @@ class SQLLoginService extends AbstractController
         return false;
     }
 
-    public function getConnection()
+    public function getConnection(): PDO
     {
         // Appel du singleton
         $sqlLogin = SQLLoginConnect::getInstance();