fix: MOVED slot redirection handler (#749)

The moved handler must update the targetSlot before tryConnection call to correctly choose the node that owns the slot.
redis · Nov 24, 2018 · bba418f · bba418f
1 parent 416f533
commit bba418f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/cluster/index.ts b/lib/cluster/index.ts
@@ -432,7 +432,7 @@ class Cluster extends EventEmitter {
       }
     }
 
-    const targetSlot = node ? node.slot : command.getSlot()
+    let targetSlot = node ? node.slot : command.getSlot()
     const ttl = {}
     const _this = this
     if (!node && !command.__is_reject_overwritten) {
@@ -443,6 +443,7 @@ class Cluster extends EventEmitter {
         _this.handleError(err, ttl, {
           moved: function (slot, key) {
             debug('command %s is moved to %s', command.name, key)
+            targetSlot = Number(slot)
             if (_this.slots[slot]) {
               _this.slots[slot][0] = key
             } else {