Skip to content
This repository was archived by the owner on May 31, 2024. It is now read-only.

Commit f5bf68f

Browse files
security #cve-2019-10911 [Security] Add a separator in the remember me cookie hash (pborreli)
This PR was merged into the 3.4 branch. Discussion ---------- [Security] Add a separator in the remember me cookie hash Based on #89 Commits ------- a29ce2817c [Security] Add a separator in the remember me cookie hash
2 parents 165f221 + 6505d7f commit f5bf68f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Http/RememberMe/TokenBasedRememberMeServices.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,6 @@ protected function generateCookieValue($class, $username, $expires, $password)
120120
*/
121121
protected function generateCookieHash($class, $username, $expires, $password)
122122
{
123-
return hash_hmac('sha256', $class.$username.$expires.$password, $this->getSecret());
123+
return hash_hmac('sha256', $class.self::COOKIE_DELIMITER.$username.self::COOKIE_DELIMITER.$expires.self::COOKIE_DELIMITER.$password, $this->getSecret());
124124
}
125125
}

0 commit comments

Comments
 (0)