Use __sleep/__wakeup instead of Serializable

Fixes symfony#38338

Author: wouterj

src/Symfony/Component/RateLimiter/LimiterStateInterface.php

@@ -12,11 +12,17 @@
 namespace Symfony\Component\RateLimiter;
 
 /**
+ * Representing the stored state of the limiter.
+ *
+ * Classes implementing this interface must be serializable,
+ * which is used by the storage implementations to store the
+ * object.
+ *
  * @author Wouter de Jong <[email protected]>
  *
  * @experimental in 5.2
  */
-interface LimiterStateInterface extends \Serializable
+interface LimiterStateInterface
 {
     public function getId(): string;
 

src/Symfony/Component/RateLimiter/TokenBucket.php

@@ -70,15 +70,22 @@ public function getExpirationTime(): int
         return $this->rate->calculateTimeForTokens($this->burstSize);
     }
 
-    public function serialize(): string
+    /**
+     * @internal
+     */
+    public function __sleep(): array
     {
-        return serialize([$this->id, $this->tokens, $this->timer, $this->burstSize, (string) $this->rate]);
+        $this->stringRate = (string) $this->rate;
+
+        return ['id', 'tokens', 'timer', 'burstSize', 'stringRate'];
     }
 
-    public function unserialize($serialized): void
+    /**
+     * @internal
+     */
+    public function __wakeup(): void
     {
-        [$this->id, $this->tokens, $this->timer, $this->burstSize, $rate] = unserialize($serialized);
-
-        $this->rate = Rate::fromString($rate);
+        $this->rate = Rate::fromString($this->stringRate);
+        unset($this->stringRate);
     }
 }

src/Symfony/Component/RateLimiter/Window.php

@@ -48,15 +48,13 @@ public function getHitCount(): int
         return $this->hitCount;
     }
 
-    public function serialize(): string
+    /**
+     * @internal
+     */
+    public function __sleep(): array
     {
         // $intervalInSeconds is not serialized, it should only be set
         // upon first creation of the Window.
-        return serialize([$this->id, $this->hitCount]);
-    }
-
-    public function unserialize($serialized): void
-    {
-        [$this->id, $this->hitCount] = unserialize($serialized);
+        return ['id', 'hitCount'];
     }
 }