fix(req-cache): ensure oldest entry is defined before deleting it

typesense · Nov 18, 2024 · 72386df · 72386df
1 parent 8d18aa6
commit 72386df
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/Typesense/RequestWithCache.ts b/src/Typesense/RequestWithCache.ts
@@ -83,13 +83,17 @@ export default class RequestWithCache {
     const isCacheOverMaxSize = this.responseCache.size > maxSize;
     if (isCacheOverMaxSize) {
       const oldestEntry = this.responseCache.keys().next().value;
-      this.responseCache.delete(oldestEntry);
+      if (oldestEntry) {
+        this.responseCache.delete(oldestEntry);
+      }
     }
     const isResponsePromiseCacheOverMaxSize =
       this.responsePromiseCache.size > maxSize;
     if (isResponsePromiseCacheOverMaxSize) {
       const oldestEntry = this.responsePromiseCache.keys().next().value;
-      this.responsePromiseCache.delete(oldestEntry);
+      if (oldestEntry) {
+        this.responsePromiseCache.delete(oldestEntry);
+      }
     }
     return response as T;
   }