From 0d0ace194a602607133e569319652c7b7b8f0e22 Mon Sep 17 00:00:00 2001 From: Graeme Fowler Date: Tue, 6 Mar 2018 11:24:10 +0000 Subject: Cheap method to follow redis cluster MOVED replies --- src/src/lookups/redis.c | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/src/lookups/redis.c b/src/src/lookups/redis.c index db408d993..c36b1bda7 100644 --- a/src/src/lookups/redis.c +++ b/src/src/lookups/redis.c @@ -241,7 +241,20 @@ switch (redis_reply->type) { case REDIS_REPLY_ERROR: *errmsg = string_sprintf("REDIS: lookup result failed: %s\n", redis_reply->str); - *defer_break = FALSE; + + /* trap MOVED cluster responses and follow them */ + if (Ustrncmp(redis_reply->str, "MOVED", 5)) + { + DEBUG(D_lookup) + debug_printf("REDIS: cluster redirect %s\n", redis_reply->str); + /* follow redirect + This is cheating, we simply set defer_break = TRUE to move on to + the next server in the redis_servers list */ + *defer_break = TRUE; + return DEFER; + } else { + *defer_break = FALSE; + } *do_cache = 0; goto REDIS_EXIT; /* NOTREACHED */ -- cgit v1.2.3