From d04ab329a111acca7996bb96d27faa79bb1341ce Mon Sep 17 00:00:00 2001
From: Lukasz Raczylo <lukasz@raczylo.com>
Date: Thu, 13 Aug 2026 02:03:50 +0100
Subject: [PATCH] fix(cache/redis): scope Clear to owned keys; drop pointless
 Info call

Clear() used FlushDB, wiping every key in the selected Redis DB even if
that database is shared with other tenants. Switch to a prefix-scoped
SCAN+DEL so only this cache's keys (prefixed) are removed, matching the
memory cache's Clear semantics.

GetMemoryUsage made an Info() round-trip purely to return 0, stalling
the admin dashboard when Redis is slow; return 0 without the call.
---
 cache/redis/redis.go | 39 ++++++++++++++++++++++++---------------
 1 file changed, 24 insertions(+), 15 deletions(-)

diff --git a/cache/redis/redis.go b/cache/redis/redis.go
index 91c05e7..ba814f3 100644
--- a/cache/redis/redis.go
+++ b/cache/redis/redis.go
@@ -77,8 +77,27 @@ func (c *RedisConfig) Delete(key string) error {
 	return c.client.Del(c.ctx, c.prependKeyName(key)).Err()
 }
 
+// Clear removes only the keys owned by this cache (those under the configured
+// prefix). Unlike FlushDB it does not wipe unrelated keys that may share the
+// selected Redis DB when the database is not exclusively owned by this proxy.
 func (c *RedisConfig) Clear() error {
-	return c.client.FlushDB(c.ctx).Err()
+	var cursor uint64
+	for {
+		keys, next, err := c.client.Scan(c.ctx, cursor, c.prependKeyName("*"), 100).Result()
+		if err != nil {
+			return err
+		}
+		if len(keys) > 0 {
+			if err := c.client.Del(c.ctx, keys...).Err(); err != nil {
+				return err
+			}
+		}
+		cursor = next
+		if cursor == 0 {
+			break
+		}
+	}
+	return nil
 }
 
 func (c *RedisConfig) CountQueries() (int64, error) {
@@ -97,21 +116,11 @@ func (c *RedisConfig) CountQueriesWithPattern(pattern string) (int, error) {
 	return len(keys), nil
 }
 
-// GetMemoryUsage returns an approximation of memory usage for Redis
-// For Redis, this is not as accurate as the memory cache implementation
-// as actual memory is managed by Redis server
+// GetMemoryUsage returns an approximation of memory usage for Redis.
+// Actual memory is managed by the Redis server, so the proxy reports 0 here.
+// Avoid an Info() round-trip: its result was always discarded, and it could
+// stall the dashboard stats endpoint when Redis is slow or unreachable.
 func (c *RedisConfig) GetMemoryUsage() int64 {
-	// We could attempt to get memory usage from Redis info
-	// but for now, we'll just return 0 since Redis manages its own memory
-	// and this information would require parsing the INFO command output
-	_, err := c.client.Info(c.ctx, "memory").Result()
-	if err != nil {
-		return 0
-	}
-
-	// Just return 0 as a placeholder since Redis manages its own memory
-	// In a production environment, you could parse the Redis INFO command result
-	// to extract actual "used_memory" value
 	return 0
 }