HttpContent buffering

7702244 · Jun 27, 2024 · 270008a · 270008a
1 parent 7e32148
commit 270008a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/src/MAOToolkit/Extensions/HttpContentExtensions.cs b/src/MAOToolkit/Extensions/HttpContentExtensions.cs
@@ -15,16 +15,12 @@ public static class HttpContentExtensions
         /// <returns>String representation of HTTP content.</returns>
         public static async Task<string> ReadAsStringAsync(this HttpContent httpContent, int charsLimit)
         {
-            if (httpContent is null)
-                throw new ArgumentNullException(nameof(httpContent));
+            ArgumentNullException.ThrowIfNull(httpContent);
 
+            await httpContent.LoadIntoBufferAsync();
+
             var stream = await httpContent.ReadAsStreamAsync();
 
-            if (!stream.CanRead)
-            {
-                return (await httpContent.ReadAsStringAsync()).Left(charsLimit);
-            }
-
             // The `leaveOpen` should be `true` if there's another function going to be invoked AFTER this.
             using var reader = new StreamReader(
                 stream,

diff --git a/src/MAOToolkit/Extensions/StreamReaderExtensions.cs b/src/MAOToolkit/Extensions/StreamReaderExtensions.cs
@@ -11,8 +11,7 @@ public static class StreamReaderExtensions
         /// <returns>String representation of readed stream.</returns>
         public static async ValueTask<string> ReadWithLimitAsync(this StreamReader reader, int charsLimit)
         {
-            if (reader is null)
-                throw new ArgumentNullException(nameof(reader));
+            ArgumentNullException.ThrowIfNull(reader);
 
             if (charsLimit > 0)
             {