Merge pull request #383 from Kuadrant/issue-378

Fix deleting secondary index from redis on limit deletion
Kuadrant · Oct 9, 2024 · 7fe2cce · 7fe2cce
2 parents ff12885 + 1e83f81
commit 7fe2cce
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/limitador/src/storage/redis/redis_async.rs b/limitador/src/storage/redis/redis_async.rs
@@ -229,6 +229,8 @@ impl AsyncRedisStorage {
                 .await?;
         }
 
+        con.del::<_, ()>(key_for_counters_of_limit(limit)).await?;
+
         Ok(())
     }
 }

diff --git a/limitador/src/storage/redis/redis_sync.rs b/limitador/src/storage/redis/redis_sync.rs
@@ -156,6 +156,7 @@ impl CounterStorage for RedisStorage {
             for counter_key in counter_keys {
                 con.del::<_, ()>(counter_key)?;
             }
+            con.del::<_, ()>(key_for_counters_of_limit(limit))?;
         }
 
         Ok(())