[Serializer] fixed fix encoding of cache keys with anonymous classes

michaelzangerle · nicolas-grekas · commit 3a4675359d7b · 2020-08-31T18:03:52.000+02:00
diff --git a/src/Symfony/Component/Serializer/Mapping/Factory/CacheClassMetadataFactory.php b/src/Symfony/Component/Serializer/Mapping/Factory/CacheClassMetadataFactory.php
@@ -44,8 +44,7 @@ public function __construct(ClassMetadataFactoryInterface $decorated, CacheItemP
     public function getMetadataFor($value)
     {
         $class = $this->getClass($value);
-        // Key cannot contain backslashes according to PSR-6
-        $key = strtr($class, '\\', '_');
+        $key = rawurlencode(strtr($class, '\\', '_'));
 
         $item = $this->cacheItemPool->getItem($key);
         if ($item->isHit()) {
diff --git a/src/Symfony/Component/Serializer/Tests/Mapping/Factory/CacheMetadataFactoryTest.php b/src/Symfony/Component/Serializer/Tests/Mapping/Factory/CacheMetadataFactoryTest.php
@@ -63,4 +63,20 @@ public function testInvalidClassThrowsException()
 
         $factory->getMetadataFor('Not\Exist');
     }
+
+    public function testAnonymousClass()
+    {
+        $anonymousObject = new class() {
+        };
+
+        $metadata = new ClassMetadata(\get_class($anonymousObject));
+        $decorated = $this->getMockBuilder(ClassMetadataFactoryInterface::class)->getMock();
+        $decorated
+            ->expects($this->once())
+            ->method('getMetadataFor')
+            ->willReturn($metadata);
+
+        $factory = new CacheClassMetadataFactory($decorated, new ArrayAdapter());
+        $this->assertEquals($metadata, $factory->getMetadataFor($anonymousObject));
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -44,8 +44,7 @@ public function __construct(ClassMetadataFactoryInterface $decorated, CacheItemP`
`44`	`44`	`public function getMetadataFor($value)`
`45`	`45`	`{`
`46`	`46`	`$class = $this->getClass($value);`
`47`		`- // Key cannot contain backslashes according to PSR-6`
`48`		`- $key = strtr($class, '\\', '_');`
	`47`	`+ $key = rawurlencode(strtr($class, '\\', '_'));`
`49`	`48`
`50`	`49`	`$item = $this->cacheItemPool->getItem($key);`
`51`	`50`	`if ($item->isHit()) {`