Fix fingerprint when context is not serializable

jderusse · jderusse · commit c4110e7b32c3 · 2021-03-11T14:57:14.000+01:00
diff --git a/src/Symfony/Bridge/Twig/Mime/BodyRenderer.php b/src/Symfony/Bridge/Twig/Mime/BodyRenderer.php
@@ -49,7 +49,7 @@ public function render(Message $message): void
 
         $previousRenderingKey = $messageContext[__CLASS__] ?? null;
         unset($messageContext[__CLASS__]);
-        $currentRenderingKey = md5(serialize([$messageContext, $message->getTextTemplate(), $message->getHtmlTemplate()]));
+        $currentRenderingKey = $this->getFingerPrint($message);
         if ($previousRenderingKey === $currentRenderingKey) {
             return;
         }
@@ -77,6 +77,23 @@ public function render(Message $message): void
         $message->context($message->getContext() + [__CLASS__ => $currentRenderingKey]);
     }
 
+    private function getFingerPrint(Message $message): string
+    {
+        $messageContext = $message->getContext();
+        unset($messageContext[__CLASS__]);
+
+        $payload = [$messageContext, $message->getTextTemplate(), $message->getHtmlTemplate()];
+        try {
+            $serialized = serialize($payload);
+        } catch (\Exception $e) {
+            // Serialization of 'Closure' is not allowed
+            // Happens when context contain a closure, in that case, we assume that context always change.
+            $serialized = uniqid('', true);
+        }
+
+        return md5($serialized);
+    }
+
     private function convertHtmlToText(string $html): string
     {
         if (null !== $this->converter) {
diff --git a/src/Symfony/Bridge/Twig/Tests/Mime/BodyRendererTest.php b/src/Symfony/Bridge/Twig/Tests/Mime/BodyRendererTest.php
@@ -100,6 +100,27 @@ public function testRenderedOnce()
         $this->assertEquals('reset', $email->getTextBody());
     }
 
+    public function testRenderedOnceUnserializableContext()
+    {
+        $twig = new Environment(new ArrayLoader([
+            'text' => 'Text',
+        ]));
+        $renderer = new BodyRenderer($twig);
+        $email = (new TemplatedEmail())
+            ->to('fabien@symfony.com')
+            ->from('helene@symfony.com')
+        ;
+        $email->textTemplate('text');
+        $email->context([
+            'foo' => static function () {
+                return 'bar';
+            },
+        ]);
+
+        $renderer->render($email);
+        $this->assertEquals('Text', $email->getTextBody());
+    }
+
     private function prepareEmail(?string $text, ?string $html, array $context = []): TemplatedEmail
     {
         $twig = new Environment(new ArrayLoader([
