1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

In HTTP parser, proxy return value of onHeadersComplete

Browse Source
This commit is contained in:
Ryan Dahl 2010-05-25 18:39:23 -07:00
parent 2fca40e44f
commit 35c14f637e
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/http.js
View File

@ -85,6 +85,8 @@ var parsers = new FreeList('parsers', 1000, function () {
// so that we can capture the first part of the new protocol // so that we can capture the first part of the new protocol
parser.onIncoming(parser.incoming, info.shouldKeepAlive); parser.onIncoming(parser.incoming, info.shouldKeepAlive);
} }
return false; // Is response to HEAD request?
}; };
parser.onBody = function (b, start, len) { parser.onBody = function (b, start, len) {

6
src/node_http_parser.cc
View File

@ -177,13 +177,13 @@ class Parser : public ObjectWrap {
Local<Value> argv[1] = { message_info }; Local<Value> argv[1] = { message_info };
Local<Value> ret = cb->Call(parser->handle_, 1, argv); Local<Value> head_response = cb->Call(parser->handle_, 1, argv);
if (ret.IsEmpty()) { if (head_response.IsEmpty()) {
parser->got_exception_ = true; parser->got_exception_ = true;
return -1; return -1;
} else { } else {
return 0; return head_response->IsTrue() ? 1 : 0;
} }
} }