Merge pull request #27 from melanger/patch-1

Log success only on success

Author: vyskocilpavel

CHANGELOG.md

@@ -2,6 +2,8 @@
 All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
+#### Fixed
+- Log info message about successful authentication only after successful authentication to SP
 
 ## [v3.1.0]
 #### Added

lib/Auth/Process/DatabaseCommand.php

@@ -73,12 +73,13 @@ public static function insertLogin(&$request, &$date)
                     "(identifier, name) VALUES (?, ?) ON DUPLICATE KEY UPDATE name = ?"
                 );
                 $stmt->bind_param("sss", $spEntityId, $spName, $spName);
-                $stmt->execute();
+                $success = $stmt->execute();
+                if ($success) {
+                    Logger::info("The login log was successfully stored in database");
+                }
             }
         }
 
-        Logger::info("The login log was successfully stored in database");
-
         $conn->close();
     }