http: remove obsolete function escapeHeaderValue

There are test cases which validate the useful path of the function never runs the functionality of it is obsoleted by checkInvalidHeaderChar PR-URL: #24173 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>

Author: Lauri Piisang

lib/_http_outgoing.js

@@ -409,7 +409,7 @@ function processHeader(self, state, key, value, validate){
 functionstoreHeader(self,state,key,value,validate){
 if(validate)
 validateHeaderValue(key,value);
-state.header+=key+': '+escapeHeaderValue(value)+CRLF;
+state.header+=key+': '+value+CRLF;
 matchHeader(self,state,key,value);
 }
 
@@ -642,13 +642,6 @@ function connectionCorkNT(msg, conn){
 }
 
 
-functionescapeHeaderValue(value){
-// Protect against response splitting. The regex test is there to
-// minimize the performance impact in the common case.
-return/[\r\n]/.test(value) ? value.replace(/[\r\n]+[\t]*/g,'') : value;
-}
-
-
 OutgoingMessage.prototype.addTrailers=functionaddTrailers(headers){
 this._trailer='';
 varkeys=Object.keys(headers);
@@ -670,7 +663,7 @@ OutgoingMessage.prototype.addTrailers = function addTrailers(headers){
 debug('Trailer "%s" contains invalid characters',field);
 thrownewERR_INVALID_CHAR('trailer content',field);
 }
-this._trailer+=field+': '+escapeHeaderValue(value)+CRLF;
+this._trailer+=field+': '+value+CRLF;
 }
 };