Allow the log level to be overriden based on the http status code

mtibben · mtibben · commit e9b545e6215b · 2017-12-18T09:32:19.000+11:00
diff --git a/src/Symfony/Component/HttpKernel/EventListener/ExceptionListener.php b/src/Symfony/Component/HttpKernel/EventListener/ExceptionListener.php
@@ -12,6 +12,7 @@
 namespace Symfony\Component\HttpKernel\EventListener;
 
 use Psr\Log\LoggerInterface;
+use Psr\Log\LogLevel;
 use Symfony\Component\Debug\Exception\FlattenException;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpKernel\Log\DebugLoggerInterface;
@@ -30,6 +31,7 @@ class ExceptionListener implements EventSubscriberInterface
 {
     protected $controller;
     protected $logger;
+    public static $httpStatusCodeLogLevel = [];
 
     public function __construct($controller, LoggerInterface $logger = null)
     {
@@ -91,11 +93,19 @@ public static function getSubscribedEvents()
     protected function logException(\Exception $exception, $message)
     {
         if (null !== $this->logger) {
-            if (!$exception instanceof HttpExceptionInterface || $exception->getStatusCode() >= 500) {
-                $this->logger->critical($message, array('exception' => $exception));
-            } else {
-                $this->logger->error($message, array('exception' => $exception));
+            $logLevel = LogLevel::ERROR;
+            if ($exception instanceof HttpExceptionInterface) {
+                $statusCode = $exception->getStatusCode();
+                if (isset(static::httpStatusCodeLogLevel[$statusCode])) {
+                    $logLevel = static::httpStatusCodeLogLevel[$statusCode];
+                } else if ($statusCode >= 500) {
+                    $logLevel = LogLevel::CRITICAL;
+                } else if ($statusCode >= 400) {
+                    $logLevel = LogLevel::WARNING;
+                }
             }
+
+            $this->logger->log($logLevel, $message, array('exception' => $exception));
         }
     }
 
