fix: Hook up the keepAlive after a successful connect

Fixes #1339
redis · Mar 30, 2022 · d63e70b · d63e70b
1 parent 92e7cd9
commit d63e70b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/Redis.ts b/lib/Redis.ts
@@ -213,10 +213,6 @@ class Redis extends Commander {
           }
 
           _this.stream = stream;
-          if (typeof options.keepAlive === "number") {
-            stream.setKeepAlive(true, options.keepAlive);
-          }
-
           if (stream.connecting) {
             stream.once(CONNECT_EVENT, eventHandler.connectHandler(_this));
 

diff --git a/lib/redis/event_handler.ts b/lib/redis/event_handler.ts
@@ -16,6 +16,10 @@ export function connectHandler(self) {
 
     self.resetCommandQueue();
 
+    if (typeof self.options.keepAlive === "number") {
+      self.stream.setKeepAlive(true, self.options.keepAlive);
+    }
+
     // AUTH command should be processed before any other commands
     let flushed = false;
     const { connectionEpoch } = self;