fixes #2524 delaying graceful exit while pingInterval is set (#2525)

* fixes #2524 * `clearTimeout` in `disconnect` too --------- Co-authored-by: Leibale Eidelman <[email protected]>
redis · Aug 23, 2023 · bc8abd5 · bc8abd5
1 parent 259e9b2
commit bc8abd5
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/packages/client/lib/client/index.ts b/packages/client/lib/client/index.ts
@@ -683,6 +683,7 @@ export default class RedisClient<
 
     QUIT(): Promise<string> {
         return this.#socket.quit(async () => {
+            if (this.#pingTimer) clearTimeout(this.#pingTimer);
             const quitPromise = this.#queue.addCommand<string>(['QUIT']);
             this.#tick();
             const [reply] = await Promise.all([
@@ -804,6 +805,7 @@ export default class RedisClient<
     }
 
     async disconnect(): Promise<void> {
+        if (this.#pingTimer) clearTimeout(this.#pingTimer);
         this.#queue.flushAll(new DisconnectsClientError());
         this.#socket.disconnect();
         await this.#destroyIsolationPool();