Remove JSON_THROW_ON_ERROR as it's a PHP 7.3 feature

amphp · Jul 7, 2019 · 7de4b9b · 7de4b9b
1 parent f83c01e
commit 7de4b9b
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/CachedResponse.php b/src/CachedResponse.php
@@ -38,7 +38,10 @@ public static function fromResponse(
     public static function fromCacheData(string $data): self
     {
         try {
-            $data = \json_decode($data, true, 4, \JSON_THROW_ON_ERROR);
+            $data = \json_decode($data, true, 4);
+            if ($data === null) {
+                throw new HttpException('Failed to decode cached data, JSON syntax error');
+            }
 
             if (!isset(
                 $data['protocol_version'],
@@ -139,7 +142,7 @@ public function toCacheData(): string
             'request_time' => $this->requestTime->getTimestamp(),
             'response_time' => $this->responseTime->getTimestamp(),
             'body_hash' => $this->bodyHash,
-        ], \JSON_THROW_ON_ERROR);
+        ]);
     }
 
     /**