Browse Source

Fix test-http-conn-reset.js on OSX

v0.7.4-release
Ryan Dahl 13 years ago
parent
commit
3a34972672
  1. 14
      lib/http.js
  2. 2
      test/simple/test-http-conn-reset.js

14
lib/http.js

@ -1071,6 +1071,14 @@ ClientRequest.prototype.abort = function() {
}
};
function createHangUpError() {
var error = new Error('socket hang up');
error.code = 'ECONNRESET';
return error;
}
ClientRequest.prototype.onSocket = function(socket) {
var req = this;
process.nextTick(function () {
@ -1128,7 +1136,7 @@ ClientRequest.prototype.onSocket = function(socket) {
if (!req.res) {
// If we don't have a response then we know that the socket
// ended prematurely and we need to emit an error on the request.
req.emit('error', new Error("Request ended prematurely."));
req.emit('error', createHangUpError());
req._hadError = true;
}
parser.finish();
@ -1148,9 +1156,7 @@ ClientRequest.prototype.onSocket = function(socket) {
// This socket error fired before we started to
// receive a response. The error needs to
// fire on the request.
var error = new Error('socket hang up');
error.code = 'ECONNRESET';
req.emit('error', error);
req.emit('error', createHangUpError());
}
}
socket.on('close', closeListener);

2
test/simple/test-http-conn-reset.js

@ -44,8 +44,8 @@ function onListen() {
assert.ok(false, 'this should never run');
});
req.on('error', function(err) {
assert.equal(err.message, 'socket hang up');
assert.equal(err.code, 'ECONNRESET');
assert.equal(err.message, 'socket hang up');
caughtError = true;
});
req.end();

Loading…
Cancel
Save