Allows Content-Length for 304 Not Modified response (#2321)

Author: alefranz

src/Kestrel.Core/Internal/Http/HttpProtocol.cs

@@ -875,6 +875,7 @@ protected void VerifyResponseContentLength()
             var responseHeaders = HttpResponseHeaders;
 
             if (!HttpMethods.IsHead(Method) &&
+                StatusCode != StatusCodes.Status304NotModified &&
                 !responseHeaders.HasTransferEncoding &&
                 responseHeaders.ContentLength.HasValue &&
                 _responseBytesWritten < responseHeaders.ContentLength.Value)

test/Kestrel.FunctionalTests/ResponseTests.cs

@@ -2662,6 +2662,35 @@ public async Task HttpsConnectionClosedWhenResponseDoesNotSatisfyMinimumDataRate
             }
         }
 
+        [Fact]
+        public async Task NonZeroContentLengthFor304StatusCodeIsAllowed()
+        {
+            using (var server = new TestServer(httpContext =>
+            {
+                var response = httpContext.Response;
+                response.StatusCode = StatusCodes.Status304NotModified;
+                response.ContentLength = 42;
+
+                return Task.CompletedTask;
+            }))
+            {
+                using (var connection = server.CreateConnection())
+                {
+                    await connection.Send(
+                        "GET / HTTP/1.1",
+                        "Host:",
+                        "",
+                        "");
+                    await connection.Receive(
+                        "HTTP/1.1 304 Not Modified",
+                        $"Date: {server.Context.DateHeaderValue}",
+                        "Content-Length: 42",
+                        "",
+                        "");
+                }
+            }
+        }
+
         public static TheoryData<string, StringValues, string> NullHeaderData
         {
             get