http: add debug message for invalid header value

This makes it easier to see what header has an invalid value. PR-URL: #9195 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]> Reviewed-By: Brian White <[email protected]>
nodejs · Feb 21, 2017 · c359018 · c359018
1 parent 72961d8
commit c359018
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/lib/_http_outgoing.js b/lib/_http_outgoing.js
@@ -315,6 +315,7 @@ function storeHeader(self, state, field, value) {
       'Header name must be a valid HTTP Token ["' + field + '"]');
   }
   if (common._checkInvalidHeaderChar(value) === true) {
+    debug('Header "%s" contains invalid characters', field);
     throw new TypeError('The header content contains invalid characters');
   }
   state.messageHeader += field + ': ' + escapeHeaderValue(value) + CRLF;
@@ -355,6 +356,7 @@ OutgoingMessage.prototype.setHeader = function(name, value) {
   if (this._header)
     throw new Error('Can\'t set headers after they are sent.');
   if (common._checkInvalidHeaderChar(value) === true) {
+    debug('Header "%s" contains invalid characters', name);
     throw new TypeError('The header content contains invalid characters');
   }
   if (this._headers === null)
@@ -532,6 +534,7 @@ OutgoingMessage.prototype.addTrailers = function(headers) {
         'Trailer name must be a valid HTTP Token ["' + field + '"]');
     }
     if (common._checkInvalidHeaderChar(value) === true) {
+      debug('Trailer "%s" contains invalid characters', field);
       throw new TypeError('The header content contains invalid characters');
     }
     this._trailer += field + ': ' + escapeHeaderValue(value) + CRLF;