bug symfony#60617 [HttpKernel] pass log level instead of exception to resolve the logger (xabbuh)

nicolas-grekas · nicolas-grekas · commit 46dca3d693b0 · 2025-06-02T08:32:14.000+02:00
This PR was merged into the 7.3 branch. Discussion ---------- [HttpKernel] pass log level instead of exception to resolve the logger | Q | A | ------------- | --- | Branch? | 7.3 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | | License | MIT Commits ------- 65a8d61 pass log level instead of exception to resolve the logger
diff --git a/src/Symfony/Component/HttpKernel/EventListener/ErrorListener.php b/src/Symfony/Component/HttpKernel/EventListener/ErrorListener.php
@@ -161,15 +161,15 @@ public static function getSubscribedEvents(): array
 
     /**
      * Logs an exception.
-     * 
+     *
      * @param ?string $logChannel
      */
     protected function logException(\Throwable $exception, string $message, ?string $logLevel = null, /* ?string $logChannel = null */): void
     {
         $logChannel = (3 < \func_num_args() ? \func_get_arg(3) : null) ?? $this->resolveLogChannel($exception);
-        
+
         $logLevel ??= $this->resolveLogLevel($exception);
-        
+
         if(!$logger = $this->getLogger($logChannel)) {
             return;
         }
@@ -218,7 +218,7 @@ protected function duplicateRequest(\Throwable $exception, Request $request): Re
         $attributes = [
             '_controller' => $this->controller,
             'exception' => $exception,
-            'logger' => DebugLoggerConfigurator::getDebugLogger($this->getLogger($exception)),
+            'logger' => DebugLoggerConfigurator::getDebugLogger($this->getLogger($this->resolveLogChannel($exception))),
         ];
         $request = $request->duplicate(null, null, $attributes);
         $request->setMethod('GET');
