fix(cache): fix limit rate cache cleanup

SocialGouv · Oct 13, 2021 · e5e17e8 · e5e17e8
1 parent a9e6367
commit e5e17e8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/.c42/share/elastic/.gitignore b/.c42/share/elastic/.gitignore
@@ -0,0 +1,2 @@
+*
+!.gitignore
diff --git a/src/server/src/middlewares/limit-rate.js b/src/server/src/middlewares/limit-rate.js
@@ -7,14 +7,13 @@ export const limitRate = ({ count, period }) => {
   const cache = new Map();
 
   const cleanup = () => {
-    cache.keys().forEach((key) => {
-      const cacheValue = cache.get(key);
-      const newCacheValue = removeExpired(cacheValue);
+    cache.forEach(([key, value]) => {
+      const newCacheValue = removeExpired(value);
 
       if (newCacheValue.length === 0) {
         cache.delete(key);
       } else {
-        cacheValue.set(key, newCacheValue);
+        cache.set(key, newCacheValue);
       }
     })
   };