From: Graeme Fowler <graemef@exim.org>
Date: Tue, 6 Mar 2018 11:24:10 +0000 (+0000)
Subject: Cheap method to follow redis cluster MOVED replies
X-Git-Tag: exim-4_91_RC1~28
X-Git-Url: https://git.exim.org/exim.git/commitdiff_plain/0d0ace194a602607133e569319652c7b7b8f0e22?ds=inline

Cheap method to follow redis cluster MOVED replies
---

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 */