[Security] Add a separator in the remember me cookie hash

pborreli · michaelcullum · commit a29ce2817cf4 · 2019-04-06T21:17:28.000+02:00
diff --git a/src/Symfony/Component/Security/Http/RememberMe/TokenBasedRememberMeServices.php b/src/Symfony/Component/Security/Http/RememberMe/TokenBasedRememberMeServices.php
@@ -120,6 +120,6 @@ protected function generateCookieValue($class, $username, $expires, $password)
      */
     protected function generateCookieHash($class, $username, $expires, $password)
     {
-        return hash_hmac('sha256', $class.$username.$expires.$password, $this->getSecret());
+        return hash_hmac('sha256', $class.self::COOKIE_DELIMITER.$username.self::COOKIE_DELIMITER.$expires.self::COOKIE_DELIMITER.$password, $this->getSecret());
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -120,6 +120,6 @@ protected function generateCookieValue($class, $username, $expires, $password)`
`120`	`120`	`*/`
`121`	`121`	`protected function generateCookieHash($class, $username, $expires, $password)`
`122`	`122`	`{`
`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());`
`124`	`124`	`}`
`125`	`125`	`}`