Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Commit

Permalink
http: fix buffer writes to outgoing messages
Browse files Browse the repository at this point in the history
  • Loading branch information
russellhaering authored and ry committed Feb 7, 2011
1 parent 71a8a3b commit e6ede31
Show file tree
Hide file tree
Showing 2 changed files with 53 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lib/http.js
Expand Up @@ -387,7 +387,7 @@ OutgoingMessage.prototype._buffer = function(data, encoding) {
var lastEncoding = this.outputEncodings[length - 1];
var lastData = this.output[length - 1];

if ((lastEncoding === encoding) ||
if ((encoding && lastEncoding === encoding) ||
(!encoding && data.constructor === lastData.constructor)) {
this.output[length - 1] = lastData + data;
return false;
Expand Down
52 changes: 52 additions & 0 deletions test/simple/test-http-client-upload-buf.js
@@ -0,0 +1,52 @@
var common = require('../common');
var assert = require('assert');
var http = require('http');

var N = 1024;
var bytesRecieved = 0;
var server_req_complete = false;
var client_res_complete = false;

var server = http.createServer(function(req, res) {
assert.equal('POST', req.method);

req.addListener('data', function(chunk) {
bytesRecieved += chunk.length;
});

req.addListener('end', function() {
server_req_complete = true;
console.log('request complete from server');
res.writeHead(200, {'Content-Type': 'text/plain'});
res.write('hello\n');
res.end();
});
});
server.listen(common.PORT);

server.addListener('listening', function() {
var client = http.createClient(common.PORT);
var req = client.request('POST', '/');

req.write(new Buffer(N));
req.end();

common.error('client finished sending request');

req.addListener('response', function(res) {
res.setEncoding('utf8');
res.addListener('data', function(chunk) {
console.log(chunk);
});
res.addListener('end', function() {
client_res_complete = true;
server.close();
});
});
});

process.addListener('exit', function() {
assert.equal(N, bytesRecieved);
assert.equal(true, server_req_complete);
assert.equal(true, client_res_complete);
});

0 comments on commit e6ede31

Please sign in to comment.