fix: Update HttpWebRequestMessage to handle MD5Stream (which does not…

… support seeking)
aws · Feb 7, 2025 · 780ed4e · 780ed4e
1 parent 5620f97
commit 780ed4e
Showing 1 changed file with 14 additions and 1 deletion.
diff --git a/sdk/src/Core/Amazon.Runtime/Pipeline/HttpHandler/_netstandard/HttpRequestMessageFactory.cs b/sdk/src/Core/Amazon.Runtime/Pipeline/HttpHandler/_netstandard/HttpRequestMessageFactory.cs
@@ -625,7 +625,20 @@ public void WriteToRequestBody(HttpContent requestContent, Stream contentStream,
             if ((isChunkedUploadWrapperStreamWithLength || isTrailingHeadersWrapperStreamWithLength || isCompressionWrapperStreamWithLength)
                 || (chunkedUploadWrapperStream == null && trailingHeadersWrapperStream == null && compressionWrapperStream == null))
             {
-                _request.Content.Headers.ContentLength = contentStream.Length - contentStream.Position;
+                long position;
+                try
+                {
+                    position = contentStream.Position;
+                }
+                catch (NotSupportedException)
+                {
+                    // If this method is invoked from an older service package, the content stream may not support getting the
+                    // current position (for example, "MD5Stream" doesn't).
+                    // In this case, we'll assume the position is 0 to maintain the previous behavior.
+                    position = 0;
+                }
+
+                _request.Content.Headers.ContentLength = contentStream.Length - position;
             }
 
             WriteContentHeaders(contentHeaders);