Skip to content
This repository was archived by the owner on May 31, 2024. It is now read-only.

Commit 4018123

Browse files
committed
Adding logging on this step and switching the order - not for any huge reason
1 parent feea383 commit 4018123

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

Guard/Firewall/GuardAuthenticationListener.php

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -165,13 +165,17 @@ public function setRememberMeServices(RememberMeServicesInterface $rememberMeSer
165165
*/
166166
private function triggerRememberMe(GuardAuthenticatorInterface $guardAuthenticator, Request $request, TokenInterface $token, Response $response = null)
167167
{
168-
if (!$guardAuthenticator->supportsRememberMe()) {
168+
if (null === $this->rememberMeServices) {
169+
if (null !== $this->logger) {
170+
$this->logger->info('Remember me skipped: it is not configured for the firewall.', array('authenticator' => get_class($guardAuthenticator)));
171+
}
172+
169173
return;
170174
}
171175

172-
if (null === $this->rememberMeServices) {
176+
if (!$guardAuthenticator->supportsRememberMe()) {
173177
if (null !== $this->logger) {
174-
$this->logger->info('Remember me skipped: it is not configured for the firewall.', array('authenticator' => get_class($guardAuthenticator)));
178+
$this->logger->info('Remember me skipped: your authenticator does not support it.', array('authenticator' => get_class($guardAuthenticator)));
175179
}
176180

177181
return;

0 commit comments

Comments
 (0)