f

Author: tmacro

lib/api/apiUtils/rateLimit/cache.js

@@ -48,14 +48,14 @@ function getCachedConfig(key) {
 
 function expireCachedConfigs(now) {
     const toRemove = [];
-    for (const [key, { expiry }] of counters.entries()) {
+    for (const [key, { expiry }] of configCache.entries()) {
         if (expiry <= now) {
             toRemove.push(key);
         }
     }
 
     for (const key of toRemove) {
-        counters.delete(key);
+        configCache.delete(key);
     }
 }
 

tests/unit/api/apiUtils/rateLimit/cache.js

@@ -100,8 +100,11 @@ describe('test limit config cache storage', () => {
             config: 10,
         });
         expireCachedConfigs(now);
-        assert.strictEqual(getCachedConfig('past'), undefined);
-        assert.strictEqual(getCachedConfig('present'), undefined);
-        assert.strictEqual(getCachedConfig('future'), 10);
+        assert.strictEqual(configCache.get('past'), undefined);
+        assert.strictEqual(configCache.get('present'), undefined);
+        assert.deepStrictEqual(configCache.get('future'), {
+            expiry: now + 10000,
+            config: 10,
+        });
     });
 });