From f26362e9381d88d4c906e3624f90fc76e9087c20 Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Fri, 1 Mar 2013 13:01:53 +0100 Subject: [PATCH] http: use socket.once, not socket.on Register the 'close' event listener with .once(), not .on(). It doesn't matter in the grand scheme of things because the listener doesn't keep references to any heavy-weight objects but using .once() for a oneshot listener is something of a best practice. --- lib/http.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/http.js b/lib/http.js index 212b8be..8fddd39 100644 --- a/lib/http.js +++ b/lib/http.js @@ -515,7 +515,7 @@ OutgoingMessage.prototype._writeRaw = function(data, encoding) { var timer = setTimeout(function() { socket.emit('close'); }); - socket.on('close', function() { + socket.once('close', function() { clearTimeout(timer); }); } -- 2.7.4