fix: reset response stream only if seekable

contensis · May 8, 2024 · d81a8bb · d81a8bb
1 parent 70fcf87
commit d81a8bb
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/src/Zengenti.Contensis.RequestHandler.Domain/ValueTypes/EndpointResponse.cs b/src/Zengenti.Contensis.RequestHandler.Domain/ValueTypes/EndpointResponse.cs
@@ -47,16 +47,20 @@ public EndpointResponse(
 
     public PageletPerformanceData? PageletPerformanceData { get; }
 
-    public Stream? ToStream(bool resetPosition = false)
+    public Stream? ToStream(bool resetPositionIfSeekable = false)
     {
         if (StreamContent != null)
         {
-            if (resetPosition)
+            if (!resetPositionIfSeekable)
             {
-                StreamContent.Position = 0;
+                return StreamContent;
             }
 
-            return StreamContent;
+            if (StreamContent.CanSeek)
+            {
+                StreamContent.Position = 0;
+                return StreamContent;
+            }
         }
 
         if (string.IsNullOrWhiteSpace(StringContent))
@@ -69,7 +73,7 @@ public EndpointResponse(
         var writer = new StreamWriter(stream, leaveOpen: true);
         writer.Write(StringContent);
         writer.Flush();
-        if (resetPosition)
+        if (resetPositionIfSeekable && stream.CanSeek)
         {
             stream.Position = 0;
         }