lib: remove double check of string type

fhinkel · jasnell · commit 35109ddaf254 · 2016-08-08T08:55:26.000-07:00
checkIsHttpToken() already checks for typeof string. We do not want to check twice. PR-URL: #7985 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Johan Bergström <bugs@bergstroem.nu> Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/lib/_http_outgoing.js b/lib/_http_outgoing.js
@@ -338,8 +338,6 @@ OutgoingMessage.prototype.setHeader = function(name, value) {
   if (!common._checkIsHttpToken(name))
     throw new TypeError(
       'Header name must be a valid HTTP Token ["' + name + '"]');
-  if (typeof name !== 'string')
-    throw new TypeError('"name" should be a string in setHeader(name, value)');
   if (value === undefined)
     throw new Error('"value" required in setHeader("' + name + '", value)');
   if (this._header)
