Skip to content
This repository was archived by the owner on May 14, 2024. It is now read-only.

Commit a8b09f3

Browse files
acappella2017jsumners
authored andcommitted
Support Node.js 10 (#497)
* Support node 10 * remove arrow function
1 parent 0eca4b6 commit a8b09f3

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

lib/client/client.js

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1114,6 +1114,12 @@ Client.prototype.connect = function connect() {
11141114
socket.once('error', bail);
11151115
socket.once('end', bail);
11161116
socket.once('timeout', bail);
1117+
socket.once('cleanupSetupListeners', function onCleanup() {
1118+
socket.removeListener('error', bail)
1119+
.removeListener('close', bail)
1120+
.removeListener('end', bail)
1121+
.removeListener('timeout', bail);
1122+
});
11171123

11181124
self._socket = socket;
11191125
self._tracker = tracker;
@@ -1150,10 +1156,8 @@ Client.prototype.connect = function connect() {
11501156

11511157
// Wire up "official" event handlers after successful connect/setup
11521158
function postSetup() {
1153-
socket.removeAllListeners('error')
1154-
.removeAllListeners('close')
1155-
.removeAllListeners('end')
1156-
.removeAllListeners('timeout');
1159+
// cleanup the listeners we attached in setup phrase.
1160+
socket.emit('cleanupSetupListeners');
11571161

11581162
// Work around lack of close event on tls.socket in node < 0.11
11591163
((socket.socket) ? socket.socket : socket).once('close',

0 commit comments

Comments
 (0)