Browse Source

Fix bug where http response.readable was never set to false

Closes GH-867.
v0.7.4-release
Abe Fettig 14 years ago
committed by Ryan Dahl
parent
commit
83727a4c86
  1. 1
      lib/http.js
  2. 19
      test/simple/test-http-response-readable.js

1
lib/http.js

@ -129,6 +129,7 @@ var parsers = new FreeList('parsers', 1000, function() {
}
if (!parser.incoming.upgrade) {
// For upgraded connections, also emit this after parser.execute
parser.incoming.readable = false;
parser.incoming.emit('end');
}
};

19
test/simple/test-http-response-readable.js

@ -0,0 +1,19 @@
var common = require('../common');
var assert = require('assert');
var http = require('http');
var testServer = new http.Server(function(req, res) {
res.writeHead(200);
res.end('Hello world');
});
testServer.listen(common.PORT, function() {
http.get({ port: common.PORT }, function(res) {
assert.equal(res.readable, true, 'res.readable initially true');
res.on('end', function() {
assert.equal(res.readable, false, 'res.readable set to false after end');
testServer.close();
});
});
});
Loading…
Cancel
Save