No need for the length limit read stream.

dotnet · Jul 24, 2023 · c99e708 · c99e708
1 parent ad03a32
commit c99e708
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/...em.Net.Http.Json/src/System/Net/Http/Json/HttpClientJsonExtensions.Get.AsyncEnumerable.cs b/...em.Net.Http.Json/src/System/Net/Http/Json/HttpClientJsonExtensions.Get.AsyncEnumerable.cs
@@ -172,14 +172,10 @@ public static partial class HttpClientJsonExtensions
                 using Stream contentStream = await HttpContentJsonExtensions.GetContentStreamAsync(
                     response.Content, cancellationToken).ConfigureAwait(false);
 
-                using LengthLimitReadStream readStream = new(contentStream, (int)client.MaxResponseContentBufferSize);
-
                 await foreach (TValue? value in JsonSerializer.DeserializeAsyncEnumerable<TValue>(
-                    readStream, jsonTypeInfo, cancellationToken).ConfigureAwait(false))
+                    contentStream, jsonTypeInfo, cancellationToken).ConfigureAwait(false))
                 {
                     yield return value;
-
-                    readStream.ResetCount();
                 }
             }
         }

diff --git a/src/libraries/System.Net.Http.Json/src/System/Net/Http/Json/LengthLimitReadStream.cs b/src/libraries/System.Net.Http.Json/src/System/Net/Http/Json/LengthLimitReadStream.cs
@@ -33,8 +33,6 @@ internal static void ThrowExceededBufferLimit(int limit)
             throw new HttpRequestException(SR.Format(SR.net_http_content_buffersize_exceeded, limit));
         }
 
-        internal void ResetCount() => _remainingLength = _lengthLimit;
-
         public override bool CanRead => _innerStream.CanRead;
         public override bool CanSeek => _innerStream.CanSeek;
         public override bool CanWrite => false;