Skip to content

Commit 0028789

Browse files
ncosta-icnilmerg
authored andcommitted
refactor(SessionStorage): add username trimming
1 parent 3be1b33 commit 0028789

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

library/Notifications/ProvidedHook/SessionStorage.php

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ public function onLogin(User $user): void
6969

7070
// cleanup existing sessions from this user (only for the current browser)
7171
$userSessions = BrowserSession::on(Database::get())
72-
->filter(Filter::equal('username', $user->getUsername()))
72+
->filter(Filter::equal('username', trim($user->getUsername())))
7373
->filter(Filter::equal('user_agent', $userAgent))
7474
->execute();
7575
/** @var BrowserSession $session */
@@ -104,7 +104,10 @@ public function onLogin(User $user): void
104104
}
105105

106106
Logger::debug(
107-
"onLogin triggered for user " . $user->getUsername() . " and browser session " . $this->session->getId()
107+
"onLogin triggered for user "
108+
. trim($user->getUsername())
109+
. " and browser session "
110+
. $this->session->getId()
108111
);
109112
}
110113
}
@@ -145,7 +148,7 @@ public function onLogout(User $user): void
145148

146149
Logger::debug(
147150
"onLogout triggered for user "
148-
. $user->getUsername()
151+
. trim($user->getUsername())
149152
. " and browser session "
150153
. $this->session->getId()
151154
);

0 commit comments

Comments
 (0)