vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: handle typical client tracking errors

Browse source
This commit is contained in:
Ankush Menat 2025-01-25 10:11:56 +05:30
parent 658e6c5e73
commit 6ea200a48f
1 changed files with 12 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
frappe/utils/redis_wrapper.py
View file

@ -10,6 +10,7 @@ from contextlib import suppress
import redis
from redis.commands.search import Search
from redis.exceptions import ResponseError
import frappe
from frappe.utils import cstr
@ -405,8 +406,17 @@ class _TrackedConnection(redis.Connection):
self.register_connect_callback(self._enable_client_tracking)
def _enable_client_tracking(self, conn):
conn.send_command("CLIENT", "TRACKING", "ON", "redirect", self._invalidator_id, "NOLOOP")
conn.read_response()
try:
conn.send_command("CLIENT", "TRACKING", "ON", "redirect", self._invalidator_id, "NOLOOP")
conn.read_response()
except ResponseError as e:
if "client ID" in str(e) and "does not exist" in str(e):
# Redis restarted, there's no easy way to recover from this.
frappe.client_cache.healthy = False
elif "unknown subcommand" in str(e).lower():
raise Exception("Redis version is not supported, upgrade to Redis 6.0 or higher.")
else:
raise
CachedValue = namedtuple("CachedValue", ["value", "expiry"])