From: Ryan Dahl <ry@tinyclouds.org>
Date: Wed, 26 May 2010 01:39:23 +0000 (-0700)
Subject: In HTTP parser, proxy return value of onHeadersComplete
X-Git-Tag: v0.1.97~24
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=35c14f637e7963bd6cc66776463339e006e60c01;p=platform%2Fupstream%2Fnodejs.git

In HTTP parser, proxy return value of onHeadersComplete
---

diff --git a/lib/http.js b/lib/http.js
index 2f46908..f46ef18 100644
--- a/lib/http.js
+++ b/lib/http.js
@@ -85,6 +85,8 @@ var parsers = new FreeList('parsers', 1000, function () {
       // so that we can capture the first part of the new protocol
       parser.onIncoming(parser.incoming, info.shouldKeepAlive);
     }
+
+    return false; // Is response to HEAD request?
   };
 
   parser.onBody = function (b, start, len) {
diff --git a/src/node_http_parser.cc b/src/node_http_parser.cc
index 3473686..29423c0 100644
--- a/src/node_http_parser.cc
+++ b/src/node_http_parser.cc
@@ -177,13 +177,13 @@ class Parser : public ObjectWrap {
 
     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;
       return -1;
     } else {
-      return 0;
+      return head_response->IsTrue() ? 1 : 0;
     }
   }