Logging middleware containers the message class and uses the message channel

sroze · sroze · commit 1202153a9bf3 · 2017-10-06T15:40:27.000+01:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/Resources/config/message.xml b/src/Symfony/Bundle/FrameworkBundle/Resources/config/message.xml
@@ -12,12 +12,6 @@
             <argument type="collection" /> <!-- Middlewares -->
         </service>
 
-        <service id="message.middleware.debug.logging" class="Symfony\Component\Message\Debug\LoggingMiddleware">
-            <argument type="service" id="logger" />
-
-            <tag name="message_middleware" priority="10" />
-        </service>
-
         <!-- Handlers -->
         <service id="message.handler_resolver" class="Symfony\Component\Message\MessageHandlerResolver">
             <argument type="collection" /> <!-- Message to handler mapping -->
@@ -46,5 +40,13 @@
 
         <service id="message.transport.default_encoder" alias="message.transport.serialize_message_with_type_in_headers" public="true" />
         <service id="message.transport.default_decoder" alias="message.transport.serialize_message_with_type_in_headers" public="true" />
+
+        <!-- Debug -->
+        <service id="message.middleware.debug.logging" class="Symfony\Component\Message\Debug\LoggingMiddleware">
+            <argument type="service" id="logger" />
+
+            <tag name="message_middleware" priority="10" />
+            <tag name="monolog.logger" channel="message" />
+        </service>
     </services>
 </container>
diff --git a/src/Symfony/Component/Message/Debug/LoggingMiddleware.php b/src/Symfony/Component/Message/Debug/LoggingMiddleware.php
@@ -34,23 +34,23 @@ public function __construct(LoggerInterface $logger)
      */
     public function handle($message, callable $next)
     {
-        $this->logger->debug('Starting processing message', array(
-            'message' => $message,
+        $this->logger->debug('Starting processing message "{message}"', array(
+            'message' => get_class($message),
         ));
 
         try {
             $result = $next($message);
         } catch (\Throwable $e) {
-            $this->logger->warning('Something went wrong while processing message', array(
-                'message' => $message,
+            $this->logger->warning('Something went wrong while processing message "{message}', array(
+                'message' => get_class($message),
                 'exception' => $e,
             ));
 
             throw $e;
         }
 
-        $this->logger->debug('Finished processing message', array(
-            'message' => $message,
+        $this->logger->debug('Finished processing message "{message}"', array(
+            'message' => get_class($message),
         ));
 
         return $result;
