Remove websocket listeners on close

Fixes #285
node-red · Jul 23, 2014 · c079576 · c079576 · dceejay · Jul 23, 2014
1 parent 5179cc7
commit c079576
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/nodes/core/io/22-websocket.js b/nodes/core/io/22-websocket.js
@@ -75,10 +75,12 @@ module.exports = function(RED) {
             // Workaround https://github.com/einaros/ws/pull/253
             // Remove listeners from RED.server
             var listener = null;
-            for (var event = 0; event < node._serverListeners.length; event++) {
-                listener = node._serverListeners[event];
-                if(typeof listener === "function"){
-                    RED.server.removeListener(event,listener);
+            for(var event in node._serverListeners) {
+                if (node._serverListeners.hasOwnProperty(event)) {
+                    listener = node._serverListeners[event];
+                    if(typeof listener === "function"){
+                        RED.server.removeListener(event,listener);
+                    }
                 }
             }
             node._serverListeners = {};