Skip to content

Commit

Permalink
StringUtils::equals() arguments in RememberMe Cookie based implementa…
Browse files Browse the repository at this point in the history
…tion are confused

It must be the other way around
  • Loading branch information
zerkms authored and fabpot committed Mar 25, 2015
1 parent 2559628 commit ea8da6e
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -54,7 +54,7 @@ protected function processAutoLoginCookie(array $cookieParts, Request $request)
throw new \RuntimeException(sprintf('The UserProviderInterface implementation must return an instance of UserInterface, but returned "%s".', get_class($user)));
}

if (true !== StringUtils::equals($hash, $this->generateCookieHash($class, $username, $expires, $user->getPassword()))) {
if (true !== StringUtils::equals($this->generateCookieHash($class, $username, $expires, $user->getPassword()), $hash)) {
throw new AuthenticationException('The cookie\'s hash is invalid.');
}

Expand Down

0 comments on commit ea8da6e

Please sign in to comment.