From: Simon Sturmer Date: Fri, 1 Jun 2012 07:33:19 +0000 (+1000) Subject: http: don't lowercase http req header until later X-Git-Tag: v0.7.10~53 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=9a998d5e244dc58a1f7650e574371e0d4c75fcc1;p=platform%2Fupstream%2Fnodejs.git http: don't lowercase http req header until later Don't lowercase the request header until we're in the _addHeaderLine method, makes it easier to intercept the raw request headers. --- diff --git a/lib/http.js b/lib/http.js index 42d72bc..92e0004 100644 --- a/lib/http.js +++ b/lib/http.js @@ -87,7 +87,7 @@ function parserOnHeadersComplete(info) { for (var i = 0; i < n; i += 2) { var k = headers[i]; var v = headers[i + 1]; - parser.incoming._addHeaderLine(k.toLowerCase(), v); + parser.incoming._addHeaderLine(k, v); } @@ -134,7 +134,7 @@ function parserOnMessageComplete() { for (var i = 0, n = headers.length; i < n; i += 2) { var k = headers[i]; var v = headers[i + 1]; - parser.incoming._addHeaderLine(k.toLowerCase(), v); + parser.incoming._addHeaderLine(k, v); } parser._headers = []; parser._url = ''; @@ -380,6 +380,7 @@ IncomingMessage.prototype._emitEnd = function() { IncomingMessage.prototype._addHeaderLine = function(field, value) { var dest = this.complete ? this.trailers : this.headers; + field = field.toLowerCase(); switch (field) { // Array headers: case 'set-cookie':