Merge branch '5.1' into 5.x

Author: derrabus

Authentication/AuthenticatorManager.php

@@ -134,8 +134,7 @@ private function executeAuthenticators(array $authenticators, Request $request):
         foreach ($authenticators as $authenticator) {
             // recheck if the authenticator still supports the listener. supports() is called
             // eagerly (before token storage is initialized), whereas authenticate() is called
-            // lazily (after initialization). This is important for e.g. the AnonymousAuthenticator
-            // as its support is relying on the (initialized) token in the TokenStorage.
+            // lazily (after initialization).
             if (false === $authenticator->supports($request)) {
                 if (null !== $this->logger) {
                     $this->logger->debug('Skipping the "{authenticator}" authenticator as it did not support the request.', ['authenticator' => \get_class($authenticator)]);

Tests/Authentication/AuthenticatorManagerTest.php

@@ -73,7 +73,7 @@ public function testSupportCheckedUponRequestAuthentication()
     {
         // the attribute stores the supported authenticators, returning false now
         // means support changed between calling supports() and authenticateRequest()
-        // (which is the case with lazy firewalls and e.g. the AnonymousAuthenticator)
+        // (which is the case with lazy firewalls)
         $authenticator = $this->createAuthenticator(false);
         $this->request->attributes->set('_security_authenticators', [$authenticator]);