diff options
author | Bernhard Schussek <bernhard.schussek@symfony-project.com> | 2011-03-17 12:34:12 +0100 |
---|---|---|
committer | Bernhard Schussek <bernhard.schussek@symfony-project.com> | 2011-03-17 12:34:12 +0100 |
commit | 2194399ddf1b18d1509aabfb744d263ac40d1ecd (patch) | |
tree | 0fc528c8ec536041017ddb805e1c1ee95d566d4a | |
parent | 68bcc66f88d402019e800639e47c6c5db16629e0 (diff) | |
parent | af0bc0a3a868f2e9f992fb7479a6b45b05087403 (diff) | |
download | symfony-security-2194399ddf1b18d1509aabfb744d263ac40d1ecd.zip symfony-security-2194399ddf1b18d1509aabfb744d263ac40d1ecd.tar.gz symfony-security-2194399ddf1b18d1509aabfb744d263ac40d1ecd.tar.bz2 |
Merge remote branch 'symfony/master' into event-manager
Conflicts:
src/Symfony/Bundle/AsseticBundle/CacheWarmer/AssetWriterCacheWarmer.php
src/Symfony/Bundle/AsseticBundle/Tests/CacheWarmer/AssetWriterCacheWarmerTest.php
src/Symfony/Bundle/FrameworkBundle/Profiler/ProfilerListener.php
src/Symfony/Bundle/FrameworkBundle/Resources/config/profiling.xml
src/Symfony/Component/HttpKernel/HttpKernel.php
src/Symfony/Component/Security/Http/Firewall/AbstractAuthenticationListener.php
src/Symfony/Component/Security/Http/Firewall/AbstractPreAuthenticatedListener.php
4 files changed, 9 insertions, 5 deletions
diff --git a/Core/Authentication/AuthenticationProviderManager.php b/Core/Authentication/AuthenticationProviderManager.php index 1d85e87..a82b9fb 100644 --- a/Core/Authentication/AuthenticationProviderManager.php +++ b/Core/Authentication/AuthenticationProviderManager.php @@ -59,6 +59,10 @@ class AuthenticationProviderManager implements AuthenticationManagerInterface try { $result = $provider->authenticate($token); + + if (null !== $result) { + break; + } } catch (AccountStatusException $e) { $e->setExtraInformation($token); diff --git a/Http/Firewall/AbstractAuthenticationListener.php b/Http/Firewall/AbstractAuthenticationListener.php index d045b7b..506d49e 100644 --- a/Http/Firewall/AbstractAuthenticationListener.php +++ b/Http/Firewall/AbstractAuthenticationListener.php @@ -105,9 +105,9 @@ abstract class AbstractAuthenticationListener implements ListenerInterface /** * Handles form based authentication. * - * @param Event $event An Event instance + * @param GetResponseEvent $event A GetResponseEvent instance */ - public function onCoreSecurity(GetResponseEvent $event) + public final function onCoreSecurity(GetResponseEvent $event) { $request = $event->getRequest(); diff --git a/Http/Firewall/AbstractPreAuthenticatedListener.php b/Http/Firewall/AbstractPreAuthenticatedListener.php index c5baa18..49039cd 100644 --- a/Http/Firewall/AbstractPreAuthenticatedListener.php +++ b/Http/Firewall/AbstractPreAuthenticatedListener.php @@ -52,7 +52,7 @@ abstract class AbstractPreAuthenticatedListener implements ListenerInterface * * @param GetResponseEvent $event A GetResponseEvent instance */ - public function onCoreSecurity(GetResponseEvent $event) + public final function onCoreSecurity(GetResponseEvent $event) { $request = $event->getRequest(); diff --git a/Http/Firewall/X509AuthenticationListener.php b/Http/Firewall/X509AuthenticationListener.php index 831ca02..2f4b78a 100644 --- a/Http/Firewall/X509AuthenticationListener.php +++ b/Http/Firewall/X509AuthenticationListener.php @@ -25,8 +25,8 @@ use Symfony\Component\EventDispatcher\EventDispatcherInterface; */ class X509AuthenticationListener extends AbstractPreAuthenticatedListener { - protected $userKey; - protected $credentialKey; + private $userKey; + private $credentialKey; public function __construct(SecurityContextInterface $securityContext, AuthenticationManagerInterface $authenticationManager, $providerKey, $userKey = 'SSL_CLIENT_S_DN_Email', $credentialKey = 'SSL_CLIENT_S_DN', LoggerInterface $logger = null, EventDispatcherInterface $dispatcher = null) { |