fix: leave response stream open after reading

contensis · May 9, 2024 · 1c2df81 · 1c2df81
1 parent e1b13e8
commit 1c2df81
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Zengenti.Contensis.RequestHandler.Application/Middleware/RequestHandlerMiddleware.cs b/src/Zengenti.Contensis.RequestHandler.Application/Middleware/RequestHandlerMiddleware.cs
@@ -415,7 +415,7 @@ private async Task HandleResponse(HttpContext context, EndpointResponse response
 
         // The content may have changed due to resolving pagelets and re-writing static paths, so get the actual length.
         var responseContent = response.ToStream(true);
-        if (responseContent is MemoryStream)
+        if (responseContent is MemoryStream && context.Response.ContentLength != responseContent.Length)
         {
             context.Response.ContentLength = responseContent.Length;
         }

diff --git a/src/Zengenti.Contensis.RequestHandler.Application/Services/EndpointRequestService.cs b/src/Zengenti.Contensis.RequestHandler.Application/Services/EndpointRequestService.cs
@@ -178,8 +178,8 @@ public async Task<EndpointResponse> Invoke(
                     var endpointResponseStream = endpointResponse.ToStream(true);
                     if (endpointResponseStream is { Length: > 0 } and MemoryStream stream)
                     {
-                        using var reader = new StreamReader(stream);
-                        responseContent = await reader.ReadToEndAsync();
+                        using var reader = new StreamReader(stream, leaveOpen: true);
+                        responseContent = await reader.ReadToEndAsync( );
                     }
                 }
             }