#259 Auto-chunk even for Connection: close responses

Author: Tratcher

src/Microsoft.Net.Http.Server/RequestProcessing/Response.cs

@@ -453,14 +453,14 @@ internal HttpApi.HTTP_FLAGS ComputeHeaders(long writeCount, bool endOfRequest =
                 _boundaryType = BoundaryType.ContentLength;
                 _expectedBodyLength = 0;
             }
-            else if (keepConnectionAlive && requestVersion == Constants.V1_1)
+            else if (requestVersion == Constants.V1_1)
             {
                 _boundaryType = BoundaryType.Chunked;
                 Headers[HttpKnownHeaderNames.TransferEncoding] = Constants.Chunked;
             }
             else
             {
-                // The length cannot be determined, so we must close the connection
+                // v1.0 and the length cannot be determined, so we must close the connection after writing data
                 keepConnectionAlive = false;
                 _boundaryType = BoundaryType.Close;
             }

test/Microsoft.AspNetCore.Server.WebListener.FunctionalTests/ResponseHeaderTests.cs

@@ -133,7 +133,8 @@ public async Task ResponseHeaders_ServerSendsConnectionClose_Closed()
                 response.EnsureSuccessStatusCode();
                 Assert.True(response.Headers.ConnectionClose.Value);
                 Assert.Equal(new string[] { "close" }, response.Headers.GetValues("Connection"));
-                Assert.False(response.Headers.TransferEncodingChunked.HasValue);
+                Assert.True(response.Headers.TransferEncodingChunked.HasValue);
+                Assert.True(response.Headers.TransferEncodingChunked);
                 IEnumerable<string> values;
                 var result = response.Content.Headers.TryGetValues("Content-Length", out values);
                 Assert.False(result);