Making all "debug" messages use the debug router

weaverryan · weaverryan · commit 5fa2684273d3 · 2015-09-26T14:16:48.000-04:00
Only the "auth success" and "auth failed" messages remain at info. That's
consistent with AbstractAuthenticationListener
diff --git a/src/Symfony/Component/Security/Guard/Firewall/GuardAuthenticationListener.php b/src/Symfony/Component/Security/Guard/Firewall/GuardAuthenticationListener.php
@@ -66,7 +66,7 @@ public function __construct(GuardAuthenticatorHandler $guardHandler, Authenticat
     public function handle(GetResponseEvent $event)
     {
         if (null !== $this->logger) {
-            $this->logger->info('Checking for guard authentication credentials.', array('firewall_key' => $this->providerKey, 'authenticators' => count($this->guardAuthenticators)));
+            $this->logger->debug('Checking for guard authentication credentials.', array('firewall_key' => $this->providerKey, 'authenticators' => count($this->guardAuthenticators)));
         }
 
         foreach ($this->guardAuthenticators as $key => $guardAuthenticator) {
@@ -77,7 +77,7 @@ public function handle(GetResponseEvent $event)
             $this->executeGuardAuthenticator($uniqueGuardKey, $guardAuthenticator, $event);
 
             if ($event->hasResponse()) {
-                $this->logger->info(sprintf('The "%s" authenticator set the response. Any later authenticator will not be called', get_class($guardAuthenticator)));
+                $this->logger->debug(sprintf('The "%s" authenticator set the response. Any later authenticator will not be called', get_class($guardAuthenticator)));
 
                 break;
             }
@@ -89,7 +89,7 @@ private function executeGuardAuthenticator($uniqueGuardKey, GuardAuthenticatorIn
         $request = $event->getRequest();
         try {
             if (null !== $this->logger) {
-                $this->logger->info('Calling getCredentials on guard configurator.', array('firewall_key' => $this->providerKey, 'authenticator' => get_class($guardAuthenticator)));
+                $this->logger->debug('Calling getCredentials() on guard configurator.', array('firewall_key' => $this->providerKey, 'authenticator' => get_class($guardAuthenticator)));
             }
 
             // allow the authenticator to fetch authentication info from the request
@@ -104,7 +104,7 @@ private function executeGuardAuthenticator($uniqueGuardKey, GuardAuthenticatorIn
             $token = new PreAuthenticationGuardToken($credentials, $uniqueGuardKey);
 
             if (null !== $this->logger) {
-                $this->logger->info('Passing guard token information to the GuardAuthenticationProvider', array('firewall_key' => $this->providerKey, 'authenticator' => get_class($guardAuthenticator)));
+                $this->logger->debug('Passing guard token information to the GuardAuthenticationProvider', array('firewall_key' => $this->providerKey, 'authenticator' => get_class($guardAuthenticator)));
             }
             // pass the token into the AuthenticationManager system
             // this indirectly calls GuardAuthenticationProvider::authenticate()
@@ -136,13 +136,13 @@ private function executeGuardAuthenticator($uniqueGuardKey, GuardAuthenticatorIn
         $response = $this->guardHandler->handleAuthenticationSuccess($token, $request, $guardAuthenticator, $this->providerKey);
         if ($response instanceof Response) {
             if (null !== $this->logger) {
-                $this->logger->info('Guard authenticator set success response.', array('response' => $response, 'authenticator' => get_class($guardAuthenticator)));
+                $this->logger->debug('Guard authenticator set success response.', array('response' => $response, 'authenticator' => get_class($guardAuthenticator)));
             }
 
             $event->setResponse($response);
         } else {
             if (null !== $this->logger) {
-                $this->logger->info('Guard authenticator set no success response: request continues.', array('authenticator' => get_class($guardAuthenticator)));
+                $this->logger->debug('Guard authenticator set no success response: request continues.', array('authenticator' => get_class($guardAuthenticator)));
             }
         }
 
@@ -173,15 +173,15 @@ private function triggerRememberMe(GuardAuthenticatorInterface $guardAuthenticat
     {
         if (null === $this->rememberMeServices) {
             if (null !== $this->logger) {
-                $this->logger->info('Remember me skipped: it is not configured for the firewall.', array('authenticator' => get_class($guardAuthenticator)));
+                $this->logger->debug('Remember me skipped: it is not configured for the firewall.', array('authenticator' => get_class($guardAuthenticator)));
             }
 
             return;
         }
 
         if (!$guardAuthenticator->supportsRememberMe()) {
             if (null !== $this->logger) {
-                $this->logger->info('Remember me skipped: your authenticator does not support it.', array('authenticator' => get_class($guardAuthenticator)));
+                $this->logger->debug('Remember me skipped: your authenticator does not support it.', array('authenticator' => get_class($guardAuthenticator)));
             }
 
             return;

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ public function __construct(GuardAuthenticatorHandler $guardHandler, Authenticat`
`66`	`66`	`public function handle(GetResponseEvent $event)`
`67`	`67`	`{`
`68`	`68`	`if (null !== $this->logger) {`
`69`		`- $this->logger->info('Checking for guard authentication credentials.', array('firewall_key' => $this->providerKey, 'authenticators' => count($this->guardAuthenticators)));`
	`69`	`+ $this->logger->debug('Checking for guard authentication credentials.', array('firewall_key' => $this->providerKey, 'authenticators' => count($this->guardAuthenticators)));`
`70`	`70`	`}`
`71`	`71`
`72`	`72`	`foreach ($this->guardAuthenticators as $key => $guardAuthenticator) {`
`@@ -77,7 +77,7 @@ public function handle(GetResponseEvent $event)`
`77`	`77`	`$this->executeGuardAuthenticator($uniqueGuardKey, $guardAuthenticator, $event);`
`78`	`78`
`79`	`79`	`if ($event->hasResponse()) {`
`80`		`- $this->logger->info(sprintf('The "%s" authenticator set the response. Any later authenticator will not be called', get_class($guardAuthenticator)));`
	`80`	`+ $this->logger->debug(sprintf('The "%s" authenticator set the response. Any later authenticator will not be called', get_class($guardAuthenticator)));`
`81`	`81`
`82`	`82`	`break;`
`83`	`83`	`}`
`@@ -89,7 +89,7 @@ private function executeGuardAuthenticator($uniqueGuardKey, GuardAuthenticatorIn`
`89`	`89`	`$request = $event->getRequest();`
`90`	`90`	`try {`
`91`	`91`	`if (null !== $this->logger) {`
`92`		`- $this->logger->info('Calling getCredentials on guard configurator.', array('firewall_key' => $this->providerKey, 'authenticator' => get_class($guardAuthenticator)));`
	`92`	`+ $this->logger->debug('Calling getCredentials() on guard configurator.', array('firewall_key' => $this->providerKey, 'authenticator' => get_class($guardAuthenticator)));`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`// allow the authenticator to fetch authentication info from the request`
`@@ -104,7 +104,7 @@ private function executeGuardAuthenticator($uniqueGuardKey, GuardAuthenticatorIn`
`104`	`104`	`$token = new PreAuthenticationGuardToken($credentials, $uniqueGuardKey);`
`105`	`105`
`106`	`106`	`if (null !== $this->logger) {`
`107`		`- $this->logger->info('Passing guard token information to the GuardAuthenticationProvider', array('firewall_key' => $this->providerKey, 'authenticator' => get_class($guardAuthenticator)));`
	`107`	`+ $this->logger->debug('Passing guard token information to the GuardAuthenticationProvider', array('firewall_key' => $this->providerKey, 'authenticator' => get_class($guardAuthenticator)));`
`108`	`108`	`}`
`109`	`109`	`// pass the token into the AuthenticationManager system`
`110`	`110`	`// this indirectly calls GuardAuthenticationProvider::authenticate()`
`@@ -136,13 +136,13 @@ private function executeGuardAuthenticator($uniqueGuardKey, GuardAuthenticatorIn`
`136`	`136`	`$response = $this->guardHandler->handleAuthenticationSuccess($token, $request, $guardAuthenticator, $this->providerKey);`
`137`	`137`	`if ($response instanceof Response) {`
`138`	`138`	`if (null !== $this->logger) {`
`139`		`- $this->logger->info('Guard authenticator set success response.', array('response' => $response, 'authenticator' => get_class($guardAuthenticator)));`
	`139`	`+ $this->logger->debug('Guard authenticator set success response.', array('response' => $response, 'authenticator' => get_class($guardAuthenticator)));`
`140`	`140`	`}`
`141`	`141`
`142`	`142`	`$event->setResponse($response);`
`143`	`143`	`} else {`
`144`	`144`	`if (null !== $this->logger) {`
`145`		`- $this->logger->info('Guard authenticator set no success response: request continues.', array('authenticator' => get_class($guardAuthenticator)));`
	`145`	`+ $this->logger->debug('Guard authenticator set no success response: request continues.', array('authenticator' => get_class($guardAuthenticator)));`
`146`	`146`	`}`
`147`	`147`	`}`
`148`	`148`
`@@ -173,15 +173,15 @@ private function triggerRememberMe(GuardAuthenticatorInterface $guardAuthenticat`
`173`	`173`	`{`
`174`	`174`	`if (null === $this->rememberMeServices) {`
`175`	`175`	`if (null !== $this->logger) {`
`176`		`- $this->logger->info('Remember me skipped: it is not configured for the firewall.', array('authenticator' => get_class($guardAuthenticator)));`
	`176`	`+ $this->logger->debug('Remember me skipped: it is not configured for the firewall.', array('authenticator' => get_class($guardAuthenticator)));`
`177`	`177`	`}`
`178`	`178`
`179`	`179`	`return;`
`180`	`180`	`}`
`181`	`181`
`182`	`182`	`if (!$guardAuthenticator->supportsRememberMe()) {`
`183`	`183`	`if (null !== $this->logger) {`
`184`		`- $this->logger->info('Remember me skipped: your authenticator does not support it.', array('authenticator' => get_class($guardAuthenticator)));`
	`184`	`+ $this->logger->debug('Remember me skipped: your authenticator does not support it.', array('authenticator' => get_class($guardAuthenticator)));`
`185`	`185`	`}`
`186`	`186`
`187`	`187`	`return;`