search

tntim96 / JSCover

JSCover is a JavaScript Code Coverage Tool that measures line, branch and function coverage
GNU General Public License v2.0
399 stars 84 forks source link

Proxy Server, Change get request first line to end with /r/n instead of /n #301

Closed abhinav1708 closed 3 years ago

abhinav1708 commented 3 years ago

As per HTTP 1.0 standard, request lines should end with CR LF i.e. \r\n https://www.w3.org/Protocols/HTTP/1.0/spec.html#Request-Line Request-Line = Method SP Request-URI SP HTTP-Version CRLF

This was being followed in handleProxyPostOrPut but for Get request line was ended with \n handleProxyRequest function line number 385 remotePrintWriter.print(method + " " + uri + " HTTP/1.0\n");

Because of this Get request was not working on some servers in proxy mode

Changed the code for Get request to end with \r\n as per standard