Browse Source

Revert "Remove 'connect' event from server side sockets"

Fixes #1276

This reverts commit f0a440d886.
v0.7.4-release
Ryan Dahl 14 years ago
parent
commit
ede1acc1ed
  1. 9
      lib/net_legacy.js
  2. 4
      test/simple/test-net-reconnect.js

9
lib/net_legacy.js

@ -955,6 +955,15 @@ function Server(/* [ options, ] listener */) {
DTRACE_NET_SERVER_CONNECTION(s); DTRACE_NET_SERVER_CONNECTION(s);
self.emit('connection', s); self.emit('connection', s);
// The 'connect' event probably should be removed for server-side
// sockets. It's redundant.
try {
s.emit('connect');
} catch (e) {
s.destroy(e);
return;
}
} }
}; };
} }

4
test/simple/test-net-reconnect.js

@ -30,7 +30,9 @@ var client_recv_count = 0;
var disconnect_count = 0; var disconnect_count = 0;
var server = net.createServer(function(socket) { var server = net.createServer(function(socket) {
socket.write('hello\r\n'); socket.addListener('connect', function() {
socket.write('hello\r\n');
});
socket.addListener('end', function() { socket.addListener('end', function() {
socket.end(); socket.end();

Loading…
Cancel
Save