summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBernhard Schussek <bernhard.schussek@symfony-project.com>2011-03-17 12:34:12 +0100
committerBernhard Schussek <bernhard.schussek@symfony-project.com>2011-03-17 12:34:12 +0100
commit2194399ddf1b18d1509aabfb744d263ac40d1ecd (patch)
tree0fc528c8ec536041017ddb805e1c1ee95d566d4a
parent68bcc66f88d402019e800639e47c6c5db16629e0 (diff)
parentaf0bc0a3a868f2e9f992fb7479a6b45b05087403 (diff)
downloadsymfony-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
-rw-r--r--Core/Authentication/AuthenticationProviderManager.php4
-rw-r--r--Http/Firewall/AbstractAuthenticationListener.php4
-rw-r--r--Http/Firewall/AbstractPreAuthenticatedListener.php2
-rw-r--r--Http/Firewall/X509AuthenticationListener.php4
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)
{