bug #10242 Missing checkPreAuth from RememberMeAuthenticationProvider

glutamatt · glutamatt · commit 30a34b0c8365 · 2014-06-30T18:57:44.000+02:00
diff --git a/src/Symfony/Component/Security/Core/Authentication/Provider/RememberMeAuthenticationProvider.php b/src/Symfony/Component/Security/Core/Authentication/Provider/RememberMeAuthenticationProvider.php
@@ -50,7 +50,7 @@ public function authenticate(TokenInterface $token)
         }
 
         $user = $token->getUser();
-        $this->userChecker->checkPostAuth($user);
+        $this->userChecker->checkPreAuth($user);
 
         $authenticatedToken = new RememberMeToken($user, $this->providerKey, $this->key);
         $authenticatedToken->setAttributes($token->getAttributes());
diff --git a/src/Symfony/Component/Security/Core/Tests/Authentication/Provider/RememberMeAuthenticationProviderTest.php b/src/Symfony/Component/Security/Core/Tests/Authentication/Provider/RememberMeAuthenticationProviderTest.php
@@ -12,7 +12,7 @@
 namespace Symfony\Component\Security\Core\Tests\Authentication\Provider;
 
 use Symfony\Component\Security\Core\Authentication\Provider\RememberMeAuthenticationProvider;
-use Symfony\Component\Security\Core\Authentication\Token\RememberMeToken;
+use Symfony\Component\Security\Core\Exception\DisabledException;
 use Symfony\Component\Security\Core\Role\Role;
 
 class RememberMeAuthenticationProviderTest extends \PHPUnit_Framework_TestCase
@@ -45,14 +45,14 @@ public function testAuthenticateWhenKeysDoNotMatch()
     }
 
     /**
-     * @expectedException \Symfony\Component\Security\Core\Exception\AccountExpiredException
+     * @expectedException \Symfony\Component\Security\Core\Exception\DisabledException
      */
-    public function testAuthenticateWhenPostChecksFails()
+    public function testAuthenticateWhenPreChecksFails()
     {
         $userChecker = $this->getMock('Symfony\Component\Security\Core\User\UserCheckerInterface');
         $userChecker->expects($this->once())
-                    ->method('checkPostAuth')
-                    ->will($this->throwException($this->getMock('Symfony\Component\Security\Core\Exception\AccountExpiredException', null, array(), '')))
+            ->method('checkPreAuth')
+            ->will($this->throwException(new DisabledException()))
         ;
 
         $provider = $this->getProvider($userChecker);

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ public function authenticate(TokenInterface $token)`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`$user = $token->getUser();`
`53`		`- $this->userChecker->checkPostAuth($user);`
	`53`	`+ $this->userChecker->checkPreAuth($user);`
`54`	`54`
`55`	`55`	`$authenticatedToken = new RememberMeToken($user, $this->providerKey, $this->key);`
`56`	`56`	`$authenticatedToken->setAttributes($token->getAttributes());`