refactor(SessionStorage): add username trimming

Icinga · Jun 12, 2024 · 0028789 · 0028789
1 parent 3be1b33
commit 0028789
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/library/Notifications/ProvidedHook/SessionStorage.php b/library/Notifications/ProvidedHook/SessionStorage.php
@@ -69,7 +69,7 @@ public function onLogin(User $user): void
 
             // cleanup existing sessions from this user (only for the current browser)
             $userSessions = BrowserSession::on(Database::get())
-                ->filter(Filter::equal('username', $user->getUsername()))
+                ->filter(Filter::equal('username', trim($user->getUsername())))
                 ->filter(Filter::equal('user_agent', $userAgent))
                 ->execute();
             /** @var BrowserSession $session */
@@ -104,7 +104,10 @@ public function onLogin(User $user): void
             }
 
             Logger::debug(
-                "onLogin triggered for user " . $user->getUsername() . " and browser session " . $this->session->getId()
+                "onLogin triggered for user "
+                . trim($user->getUsername())
+                . " and browser session "
+                . $this->session->getId()
             );
         }
     }
@@ -145,7 +148,7 @@ public function onLogout(User $user): void
 
             Logger::debug(
                 "onLogout triggered for user "
-                . $user->getUsername()
+                . trim($user->getUsername())
                 . " and browser session "
                 . $this->session->getId()
             );