fix(symfony): context not serializable when session (#6302)

MariusJam · soyuka · soyuka · commit 9ac0062a2d30 · 2024-04-13T08:51:46.000+02:00
* fix(symfony): context not serializable when session

* phpstan

---------

Co-authored-by: soyuka &lt;soyuka@users.noreply.github.com&gt;
diff --git a/src/Symfony/Messenger/ContextStamp.php b/src/Symfony/Messenger/ContextStamp.php
@@ -13,6 +13,7 @@
 
 namespace ApiPlatform\Symfony\Messenger;
 
+use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\Messenger\Stamp\StampInterface;
 
 /**
@@ -22,8 +23,15 @@
  */
 final class ContextStamp implements StampInterface
 {
-    public function __construct(private readonly array $context = [])
+    private readonly array $context;
+
+    public function __construct(array $context = [])
     {
+        if (($request = ($context['request'] ?? null)) && $request instanceof Request && $request->hasSession()) {
+            unset($context['request']);
+        }
+
+        $this->context = $context;
     }
 
     /**
diff --git a/tests/Symfony/Messenger/ContextStampTest.php b/tests/Symfony/Messenger/ContextStampTest.php
@@ -15,6 +15,7 @@
 
 use ApiPlatform\Symfony\Messenger\ContextStamp;
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\Messenger\Stamp\StampInterface;
 
 /**
@@ -32,4 +33,16 @@ public function testGetContext(): void
         $contextStamp = new ContextStamp();
         $this->assertIsArray($contextStamp->getContext());
     }
+
+    /**
+     * @doesNotPerformAssertions
+     */
+    public function testSerializable(): void
+    {
+        $request = new Request();
+        $request->setSessionFactory(function (): void {}); // @phpstan-ignore-line
+
+        $stamp = new ContextStamp(['request' => $request]);
+        serialize($stamp);
+    }
 }
