Do not schedule ping timer if the interval and timeout not initialised

socketio · Sep 18, 2024 · da2fa2b · da2fa2b
1 parent 37cbb8d
commit da2fa2b
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/packages/engine.io-client/lib/socket.ts b/packages/engine.io-client/lib/socket.ts
@@ -633,6 +633,8 @@ export class SocketWithoutUpgrade extends Emitter<
    */
   private _resetPingTimeout() {
     this.clearTimeoutFn(this._pingTimeoutTimer);
+    if (this._pingInterval <= 0 || this._pingTimeout <= 0) return;
+
     const delay = this._pingInterval + this._pingTimeout;
     this._pingTimeoutTime = Date.now() + delay;
     this._pingTimeoutTimer = this.setTimeoutFn(() => {
@@ -723,7 +725,7 @@ export class SocketWithoutUpgrade extends Emitter<
    * @return {boolean}
    */
   public isResponsive() {
-    if (this.readyState === 'closed') return false;
+    if (this.readyState === "closed") return false;
     if (this._pingTimeoutTime === null) return true;
 
     const responsive = Date.now() < this._pingTimeoutTime;