Merge pull request #92 from davemcdermid/empty-response-first-line

justcoding121 · web-flow · commit c2ade782ad6a · 2016-07-05T21:19:02.000-04:00
Fix to handle empty first-line in HTTP Response
diff --git a/Titanium.Web.Proxy/Http/HttpWebClient.cs b/Titanium.Web.Proxy/Http/HttpWebClient.cs
@@ -115,8 +115,10 @@ internal async Task ReceiveResponse()
 
             if (string.IsNullOrEmpty(httpResult[0]))
             {
-                await ServerConnection.StreamReader.ReadLineAsync();
+                //Empty content in first-line, try again
+                httpResult = (await ServerConnection.StreamReader.ReadLineAsync()).Split(ProxyConstants.SpaceSplit, 3);
             }
+
             var httpVersion = httpResult[0].Trim().ToLower();
 
             var version = new Version(1,1);

Original file line number	Diff line number	Diff line change
`@@ -115,8 +115,10 @@ internal async Task ReceiveResponse()`
`115`	`115`
`116`	`116`	`if (string.IsNullOrEmpty(httpResult[0]))`
`117`	`117`	`{`
`118`		`- await ServerConnection.StreamReader.ReadLineAsync();`
	`118`	`+ //Empty content in first-line, try again`
	`119`	`+ httpResult = (await ServerConnection.StreamReader.ReadLineAsync()).Split(ProxyConstants.SpaceSplit, 3);`
`119`	`120`	`}`
	`121`	`+`
`120`	`122`	`var httpVersion = httpResult[0].Trim().ToLower();`
`121`	`123`
`122`	`124`	`var version = new Version(1,1);`