fix #2010 - stop reconnect after .disconnect() (#2323)

* fix #2010 - stop reconnect after .disconnect() * fix quit
redis · Nov 24, 2022 · 13ad249 · 13ad249
1 parent 28b9701
commit 13ad249
Showing 1 changed file with 18 additions and 8 deletions.
diff --git a/packages/client/lib/client/socket.ts b/packages/client/lib/client/socket.ts
@@ -105,6 +105,7 @@ export default class RedisSocket extends EventEmitter {
             throw new Error('Socket already opened');
         }
 
+        this.#isOpen = true;
         return this.#connect();
     }
 
@@ -116,7 +117,6 @@ export default class RedisSocket extends EventEmitter {
             }
 
             try {
-                this.#isOpen = true;
                 this.#socket = await this.#createSocket();
                 this.#writableNeedDrain = false;
                 this.emit('connect');
@@ -142,7 +142,7 @@ export default class RedisSocket extends EventEmitter {
                 await promiseTimeout(retryIn);
             }
             retries++;
-        } while (!this.#isReady);
+        } while (this.#isOpen && !this.#isReady);
     }
 
     #createSocket(): Promise<net.Socket | tls.TLSSocket> {
@@ -203,6 +203,8 @@ export default class RedisSocket extends EventEmitter {
         this.#isReady = false;
         this.emit('error', err);
 
+        if (!this.#isOpen) return;
+
         this.#connect(true).catch(() => {
             // the error was already emitted, silently ignore it
         });
@@ -219,14 +221,22 @@ export default class RedisSocket extends EventEmitter {
     }
 
     disconnect(): void {
-        if (!this.#socket) {
+        if (!this.#isOpen) {
             throw new ClientClosedError();
-        } else {
-            this.#isOpen = this.#isReady = false;
         }
 
-        this.#socket.destroy();
-        this.#socket = undefined;
+        this.#isOpen = false;
+        this.#disconnect();
+    }
+
+    #disconnect(): void {
+        this.#isReady = false;
+
+        if (this.#socket) {
+            this.#socket.destroy();
+            this.#socket = undefined;
+        }
+
         this.emit('end');
     }
 
@@ -237,7 +247,7 @@ export default class RedisSocket extends EventEmitter {
 
         this.#isOpen = false;
         await fn();
-        this.disconnect();
+        this.#disconnect();
     }
 
     #isCorked = false;