--wip-- [skip ci]

Author: nkaradzhov

packages/client/lib/client/commands-queue.ts

@@ -65,6 +65,7 @@ export default class RedisCommandsQueue {
   }
 
   #invalidateCallback?: (key: RedisArgument | null) => unknown;
+  #movingCallback?: (afterMs: number, host: string, port: number) => void;
 
   constructor(
     respVersion: RespVersions,
@@ -134,6 +135,19 @@ export default class RedisCommandsQueue {
               }
               break;
             }
+            case 'MOVING': {
+              if (this.#movingCallback) {
+                console.log('received moving', push)
+                const [_, afterMs, url] = push;
+                let [host, port] = url.toString().split(':');
+                //['18.200.246.58'] - for some reason the server sends the host this way
+                if(host.includes('[')) {
+                  host = host.slice(2, -2);
+                }
+                this.#movingCallback(afterMs, host, Number(port));
+              }
+              break;
+            }
           }
         }
       },
@@ -145,6 +159,10 @@ export default class RedisCommandsQueue {
     this.#invalidateCallback = callback;
   }
 
+  setMovingCallback(callback?: (afterMs: number, host: string, port: number) => void) {
+    this.#movingCallback = callback;
+  }
+
   addCommand<T>(
     args: ReadonlyArray<RedisArgument>,
     options?: CommandOptions

packages/client/lib/client/index.ts

@@ -390,7 +390,7 @@ export default class RedisClient<
   }
 
   readonly #options?: RedisClientOptions<M, F, S, RESP, TYPE_MAPPING>;
-  readonly #socket: RedisSocket;
+  #socket: RedisSocket;
   readonly #queue: RedisCommandsQueue;
   #selectedDB = 0;
   #monitorCallback?: MonitorCallback<TYPE_MAPPING>;
@@ -455,7 +455,26 @@ export default class RedisClient<
     this.#validateOptions(options)
     this.#options = this.#initiateOptions(options);
     this.#queue = this.#initiateQueue();
-    this.#socket = this.#initiateSocket();
+    this.#socket = this.#initiateSocket(this.#options);
+
+    this.#queue.setMovingCallback(async (afterMs: number, host: string, port: number) => {
+      console.log(`Moving to ${host}:${port} before ${afterMs}ms`);
+      const oldSocket = this.#socket;
+      const newSocket = this.#initiateSocket({
+        ...this.#options,
+        socket: {
+          ...this.#options?.socket,
+          host,
+          port
+        }
+      });
+      newSocket.on('ready', () => {
+        console.log(`Connected to ${host}:${port}, destroying old socket`);
+        oldSocket.destroy()
+        this.#socket = newSocket
+      });
+      await newSocket.connect()
+    });
 
     if (options?.clientSideCache) {
       if (options.clientSideCache instanceof ClientSideCacheProvider) {
@@ -676,8 +695,9 @@ export default class RedisClient<
     return commands;
   }
 
-  #initiateSocket(): RedisSocket {
+  #initiateSocket(options?: RedisClientOptions<M, F, S, RESP, TYPE_MAPPING>): RedisSocket {
     const socketInitiator = async () => {
+      console.log('Initiator...');
       const promises = [],
         chainId = Symbol('Socket Initiator');
 
@@ -707,8 +727,9 @@ export default class RedisClient<
       }
     };
 
-    return new RedisSocket(socketInitiator, this.#options?.socket)
+    return new RedisSocket(socketInitiator, options?.socket)
       .on('data', chunk => {
+        console.log('Data received', chunk);
         try {
           this.#queue.decoder.write(chunk);
         } catch (err) {
@@ -717,6 +738,7 @@ export default class RedisClient<
         }
       })
       .on('error', err => {
+        console.error('Socket error', err);
         this.emit('error', err);
         this.#clientSideCache?.onError();
         if (this.#socket.isOpen && !this.#options?.disableOfflineQueue) {
@@ -727,6 +749,7 @@ export default class RedisClient<
       })
       .on('connect', () => this.emit('connect'))
       .on('ready', () => {
+        console.log('Socket ready');
         this.emit('ready');
         this.#setPingTimer();
         this.#maybeScheduleWrite();

packages/client/lib/client/socket.ts

@@ -209,6 +209,7 @@ export default class RedisSocket extends EventEmitter {
   }
 
   async #connect(): Promise<void> {
+    console.log('Connecting...');
     let retries = 0;
     do {
       try {
@@ -224,6 +225,7 @@ export default class RedisSocket extends EventEmitter {
         }
         this.#isReady = true;
         this.#socketEpoch++;
+        console.log('Socket connected, emit ready');
         this.emit('ready');
       } catch (err) {
         const retryIn = this.#shouldReconnect(retries++, err as Error);