Merge pull request #23 from TechBuddyTR/bug/Validation-ReReadingBodyObject

salihcantekin · web-flow · commit 1681adf20b8d · 2022-11-24T17:16:21.000Z
Bug/validation re reading body object
diff --git a/readme.md b/readme.md
@@ -29,8 +29,8 @@ Not only it will be only a single line of code to use them with default configs,
 
 | Package Name | Package | Download |
 | ------------- | ------------- | ------------- |
-| ApiVersioning | [![ApiVersioningNuget](https://img.shields.io/nuget/v/TechBuddy.Extensions.AspNetCore.ApiVersioning?style=for-the-badge)](https://www.nuget.org/packages/TechBuddy.Extensions.AspNetCore.ApiVersioning) | [![Downloads](https://img.shields.io/nuget/dt/TechBuddy.Extensions.AspNetCore.ApiVersioning?style=for-the-badge)](https://www.nuget.org/packages/TechBuddy.Extensions.AspNetCore.ApiVersioning/) |
-| Validation | [![ValidationNuget](https://img.shields.io/nuget/v/TechBuddy.Extensions.Validation?style=for-the-badge)](https://www.nuget.org/packages/TechBuddy.Extensions.Validation) | [![Downloads](https://img.shields.io/nuget/dt/TechBuddy.Extensions.Validation?style=for-the-badge)](https://www.nuget.org/packages/TTechBuddy.Extensions.Validation/) |
+| ApiVersioning | [![](https://img.shields.io/nuget/v/TechBuddy.Extensions.AspNetCore.ApiVersioning?style=for-the-badge)](https://www.nuget.org/packages/TechBuddy.Extensions.AspNetCore.ApiVersioning) | [![](https://img.shields.io/nuget/dt/TechBuddy.Extensions.AspNetCore.ApiVersioning?style=for-the-badge)](https://www.nuget.org/packages/TechBuddy.Extensions.AspNetCore.ApiVersioning/) |
+| Validation | [![](https://img.shields.io/nuget/v/TechBuddy.Extensions.Validation?style=for-the-badge)](https://www.nuget.org/packages/TechBuddy.Extensions.Validation) | [![](https://img.shields.io/nuget/dt/TechBuddy.Extensions.Validation?style=for-the-badge)](https://www.nuget.org/packages/TTechBuddy.Extensions.Validation/) |
 
 
 
diff --git a/src/ValidationExtension/ValidationExtension/Extensions/HttpRequestExtensions.cs b/src/ValidationExtension/ValidationExtension/Extensions/HttpRequestExtensions.cs
@@ -53,9 +53,11 @@ public static async Task<ValidationResultModel<TModel>> ValidateAsync<TModel, TV
     /// <returns>The generic object</returns>
     private static async Task<T> ReadBodyAsync<T>(this HttpRequest req)
     {
-        using var reader = new StreamReader(req.Body, Encoding.UTF8);
-
+        req.EnableBuffering();
+        req.Body.Seek(0, SeekOrigin.Begin);
+        using var reader = new StreamReader(req.Body, Encoding.UTF8, leaveOpen: true);
         var jsonBody = await reader.ReadToEndAsync();
+        req.Body.Position = 0;
 
         return JsonSerializer.Deserialize<T>(jsonBody);
     }

Original file line number	Diff line number	Diff line change
`@@ -53,9 +53,11 @@ public static async Task<ValidationResultModel<TModel>> ValidateAsync<TModel, TV`
`53`	`53`	`/// <returns>The generic object</returns>`
`54`	`54`	`private static async Task<T> ReadBodyAsync<T>(this HttpRequest req)`
`55`	`55`	`{`
`56`		`- using var reader = new StreamReader(req.Body, Encoding.UTF8);`
`57`		`-`
	`56`	`+ req.EnableBuffering();`
	`57`	`+ req.Body.Seek(0, SeekOrigin.Begin);`
	`58`	`+ using var reader = new StreamReader(req.Body, Encoding.UTF8, leaveOpen: true);`
`58`	`59`	`var jsonBody = await reader.ReadToEndAsync();`
	`60`	`+ req.Body.Position = 0;`
`59`	`61`
`60`	`62`	`return JsonSerializer.Deserialize<T>(jsonBody);`
`61`	`63`	`}`