+ stop connection interval upon different client termination events

Author: Nick Schwab

lib/node/dist/index.js

@@ -64,6 +64,7 @@ class WebSocketServer extends TransportBase {
         // Event listener for a closed connection
         client.on('close', () => {
             console.log('server received close event');
+            this._stopConnectionLostInterval();
             if (this._transportCallback !== null) {
                 this._transportCallback.onConnectionTerminated();
             }
@@ -93,6 +94,7 @@ class WebSocketServer extends TransportBase {
             this._connectionLostInterval = setInterval(() => {
                 if (client.isAlive === false) {
                     console.log('server deemed client dead');
+                    this._stopConnectionLostInterval();
                     return client.terminate();
                 }
 
@@ -115,10 +117,7 @@ class WebSocketServer extends TransportBase {
     stop () {
         const client = this._transportConfig.getClient();
 
-        if (this._connectionLostInterval !== null) {
-            clearInterval(this._connectionLostInterval);
-        }
-
+        this._stopConnectionLostInterval();
         client.terminate();
 
         return this;
@@ -137,6 +136,19 @@ class WebSocketServer extends TransportBase {
         return this;
     }
 
+    /**
+     * Stop the connection lost interval if one is defined
+     * @return {WebSocketServer}
+     * @private
+     */
+    _stopConnectionLostInterval () {
+        if (this._connectionLostInterval !== null) {
+            clearInterval(this._connectionLostInterval);
+        }
+
+        return this;
+    }
+
     /**
      * This is called whenever a new message comes in
      * @param {MessageEvent} message