summaryrefslogtreecommitdiffstats
path: root/Core/Authentication/Provider
diff options
context:
space:
mode:
authorPaweł Wacławczyk <p.a.waclawczyk@gmail.com>2013-09-11 21:24:30 +0200
committerFabien Potencier <fabien.potencier@gmail.com>2013-12-29 16:53:42 +0100
commit3e020d61729721b526c02c54f8cd6894c8ac9bd4 (patch)
treecbf0f25d789c881ebf09ce444561fa863edb809a /Core/Authentication/Provider
parent21e1cf7af986cce991eb77460765a98181936815 (diff)
downloadsymfony-security-3e020d61729721b526c02c54f8cd6894c8ac9bd4.zip
symfony-security-3e020d61729721b526c02c54f8cd6894c8ac9bd4.tar.gz
symfony-security-3e020d61729721b526c02c54f8cd6894c8ac9bd4.tar.bz2
[Security] Fixed problem with losing ROLE_PREVIOUS_ADMIN role.
Diffstat (limited to 'Core/Authentication/Provider')
-rw-r--r--Core/Authentication/Provider/UserAuthenticationProvider.php26
1 files changed, 25 insertions, 1 deletions
diff --git a/Core/Authentication/Provider/UserAuthenticationProvider.php b/Core/Authentication/Provider/UserAuthenticationProvider.php
index 626f50b..18c3e70 100644
--- a/Core/Authentication/Provider/UserAuthenticationProvider.php
+++ b/Core/Authentication/Provider/UserAuthenticationProvider.php
@@ -19,6 +19,7 @@ use Symfony\Component\Security\Core\Exception\BadCredentialsException;
use Symfony\Component\Security\Core\Exception\AuthenticationServiceException;
use Symfony\Component\Security\Core\Authentication\Token\UsernamePasswordToken;
use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
+use Symfony\Component\Security\Core\Role\SwitchUserRole;
/**
* UserProviderInterface retrieves users for UsernamePasswordToken tokens.
@@ -92,7 +93,7 @@ abstract class UserAuthenticationProvider implements AuthenticationProviderInter
throw $e;
}
- $authenticatedToken = new UsernamePasswordToken($user, $token->getCredentials(), $this->providerKey, $user->getRoles());
+ $authenticatedToken = new UsernamePasswordToken($user, $token->getCredentials(), $this->providerKey, $this->getRoles($user, $token));
$authenticatedToken->setAttributes($token->getAttributes());
return $authenticatedToken;
@@ -107,6 +108,29 @@ abstract class UserAuthenticationProvider implements AuthenticationProviderInter
}
/**
+ * Retrieves roles from user and appends SwitchUserRole if original token contained one.
+ *
+ * @param UserInterface $user The user
+ * @param TokenInterface $token The token
+ *
+ * @return Role[] The user roles
+ */
+ private function getRoles(UserInterface $user, TokenInterface $token)
+ {
+ $roles = $user->getRoles();
+
+ foreach ($token->getRoles() as $role) {
+ if ($role instanceof SwitchUserRole) {
+ $roles[] = $role;
+
+ break;
+ }
+ }
+
+ return $roles;
+ }
+
+ /**
* Retrieves the user from an implementation-specific location.
*
* @param string $username The username to retrieve