[Messenger] Do not log the message object itself

ajardin · ajardin · commit f1604e603674 · 2022-06-22T09:26:05.000+02:00
In order to avoid the leak of sensitive data (e.g. credentials) or the overflow of third-party services.
diff --git a/src/Symfony/Component/Messenger/EventListener/SendFailedMessageForRetryListener.php b/src/Symfony/Component/Messenger/EventListener/SendFailedMessageForRetryListener.php
@@ -56,7 +56,6 @@ public function onMessageFailed(WorkerMessageFailedEvent $event)
 
         $message = $envelope->getMessage();
         $context = [
-            'message' => $message,
             'class' => \get_class($message),
         ];
 
diff --git a/src/Symfony/Component/Messenger/Middleware/HandleMessageMiddleware.php b/src/Symfony/Component/Messenger/Middleware/HandleMessageMiddleware.php
@@ -53,7 +53,6 @@ public function handle(Envelope $envelope, StackInterface $stack): Envelope
         $message = $envelope->getMessage();
 
         $context = [
-            'message' => $message,
             'class' => \get_class($message),
         ];
 
diff --git a/src/Symfony/Component/Messenger/Middleware/SendMessageMiddleware.php b/src/Symfony/Component/Messenger/Middleware/SendMessageMiddleware.php
@@ -46,7 +46,6 @@ public function __construct(SendersLocatorInterface $sendersLocator, EventDispat
     public function handle(Envelope $envelope, StackInterface $stack): Envelope
     {
         $context = [
-            'message' => $envelope->getMessage(),
             'class' => \get_class($envelope->getMessage()),
         ];
 
diff --git a/src/Symfony/Component/Messenger/Worker.php b/src/Symfony/Component/Messenger/Worker.php
@@ -208,7 +208,6 @@ private function ack(): bool
             if (null !== $this->logger) {
                 $message = $envelope->getMessage();
                 $context = [
-                    'message' => $message,
                     'class' => \get_class($message),
                 ];
                 $this->logger->info('{class} was handled successfully (acknowledging to transport).', $context);

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,6 @@ public function __construct(SendersLocatorInterface $sendersLocator, EventDispat`
`46`	`46`	`public function handle(Envelope $envelope, StackInterface $stack): Envelope`
`47`	`47`	`{`
`48`	`48`	`$context = [`
`49`		`- 'message' => $envelope->getMessage(),`
`50`	`49`	`'class' => \get_class($envelope->getMessage()),`
`51`	`50`	`];`
`52`	`51`