Skip to content

Commit 01fc5b3

Browse files
committed
[Security] Remove dead references to AnonymousAuthenticator
1 parent 2393b3e commit 01fc5b3

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

Authentication/AuthenticatorManager.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,8 +129,7 @@ private function executeAuthenticators(array $authenticators, Request $request):
129129
foreach ($authenticators as $authenticator) {
130130
// recheck if the authenticator still supports the listener. supports() is called
131131
// eagerly (before token storage is initialized), whereas authenticate() is called
132-
// lazily (after initialization). This is important for e.g. the AnonymousAuthenticator
133-
// as its support is relying on the (initialized) token in the TokenStorage.
132+
// lazily (after initialization).
134133
if (false === $authenticator->supports($request)) {
135134
if (null !== $this->logger) {
136135
$this->logger->debug('Skipping the "{authenticator}" authenticator as it did not support the request.', ['authenticator' => \get_class($authenticator)]);

Tests/Authentication/AuthenticatorManagerTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ public function testSupportCheckedUponRequestAuthentication()
7171
{
7272
// the attribute stores the supported authenticators, returning false now
7373
// means support changed between calling supports() and authenticateRequest()
74-
// (which is the case with lazy firewalls and e.g. the AnonymousAuthenticator)
74+
// (which is the case with lazy firewalls)
7575
$authenticator = $this->createAuthenticator(false);
7676
$this->request->attributes->set('_security_authenticators', [$authenticator]);
7777

0 commit comments

Comments
 (0)