Catch exception when unserializing

Setono · Aug 15, 2022 · 9463319 · 9463319
1 parent 72c6e91
commit 9463319
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/TagBag.php b/src/TagBag.php
@@ -19,6 +19,7 @@
 use Setono\TagBag\Tag\RenderedTag;
 use Setono\TagBag\Tag\TagInterface;
 use Throwable;
+use TypeError;
 
 final class TagBag implements TagBagInterface, LoggerAwareInterface
 {
@@ -195,7 +196,11 @@ public function restore(): void
 
         $this->tags = [];
         if (null !== $data) {
-            $this->tags = $this->unserialize($data);
+            try {
+                $this->tags = $this->unserialize($data);
+            } catch (TypeError $e) {
+                $this->logger->error(sprintf('Exception thrown when trying to unserialize data. Error was: %s. Data was: %s', $e->getMessage(), $data));
+            }
         }
     }
 
@@ -242,6 +247,8 @@ private function findTagByFingerprint(string $fingerprint): ?array
     /**
      * @psalm-suppress MixedInferredReturnType
      *
+     * @throws TypeError if the unserialized data doesn't unserialize to instances of RenderedTag
+     *
      * @return array<string, list<RenderedTag>>
      */
     private function unserialize(string $data): array