This changes the behavior for http to send send a Content-Length header instead of using chunked encoding when we know the size of the body when sending the headers. Fixes: https://github.com/iojs/io.js/issues/1044 PR-URL: https://github.com/iojs/io.js/pull/1062 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Brian White <mscdex@mscdex.net>
39 lines
1,003 B
JavaScript
39 lines
1,003 B
JavaScript
var common = require('../common');
|
|
var assert = require('assert');
|
|
|
|
var http = require('http');
|
|
|
|
var server = http.createServer(function(request, response) {
|
|
// removed headers should stay removed, even if node automatically adds them
|
|
// to the output:
|
|
response.removeHeader('connection');
|
|
response.removeHeader('transfer-encoding');
|
|
response.removeHeader('content-length');
|
|
|
|
// make sure that removing and then setting still works:
|
|
response.removeHeader('date');
|
|
response.setHeader('date', 'coffee o clock');
|
|
|
|
response.end('beep boop\n');
|
|
|
|
this.close();
|
|
});
|
|
|
|
var response = '';
|
|
|
|
process.on('exit', function() {
|
|
assert.equal('beep boop\n', response);
|
|
console.log('ok');
|
|
});
|
|
|
|
server.listen(common.PORT, function() {
|
|
http.get({ port: common.PORT }, function(res) {
|
|
assert.equal(200, res.statusCode);
|
|
assert.deepEqual(res.headers, { date : 'coffee o clock' });
|
|
|
|
res.setEncoding('ascii');
|
|
res.on('data', function(chunk) {
|
|
response += chunk;
|
|
});
|
|
});
|
|
});
|