Fix edge case issues with Redis

Closes #103
GrahamCampbell · Dec 4, 2023 · 927527d · 927527d
1 parent fadb939
commit 927527d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@ CHANGE LOG
 ## V10.1 (04/12/2023)
 
 * Added PHP 8.3 support
+* Fix edge case issues with Redis
 
 
 ## V10.0 (07/03/2023)

diff --git a/src/Throttler/CacheThrottler.php b/src/Throttler/CacheThrottler.php
@@ -138,10 +138,6 @@ public function count(): int
 
         $this->number = (int) $this->store->get($this->key);
 
-        if (!$this->number) {
-            $this->number = 0;
-        }
-
         return $this->number;
     }
 
@@ -166,7 +162,7 @@ private function hitRedis(): void
                'if v>1 then return v '.
                'else redis.call(\'setex\', KEYS[1], ARGV[1], 1) return 1 end';
 
-        $this->number = $this->store->connection()->eval($lua, 1, $this->computeRedisKey(), $this->time);
+        $this->number = (int) $this->store->connection()->eval($lua, 1, $this->computeRedisKey(), $this->time);
     }
 
     /**