fix: 🐛 bug fix in bad request exception

mehdihadeli · Jun 15, 2023 · c25e4d6 · c25e4d6
1 parent c6d0045
commit c25e4d6
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 7 deletions.
diff --git a/src/Vertical.Slice.Template.Shared/Core/Exceptions/BadRequestException.cs b/src/Vertical.Slice.Template.Shared/Core/Exceptions/BadRequestException.cs
@@ -5,5 +5,5 @@ namespace Vertical.Slice.Template.Shared.Core.Exceptions;
 public class BadRequestException : CustomException
 {
     public BadRequestException(string message, Exception? innerException = null)
-        : base(message, StatusCodes.Status404NotFound, innerException) { }
+        : base(message, StatusCodes.Status400BadRequest, innerException) { }
 }
diff --git a/...blemDetail/Middlewares/CaptureExceptionMiddleware/CaptureExceptionMiddlewareExtensions.cs b/...blemDetail/Middlewares/CaptureExceptionMiddleware/CaptureExceptionMiddlewareExtensions.cs
@@ -2,6 +2,7 @@
 
 namespace Vertical.Slice.Template.Shared.Web.ProblemDetail.Middlewares.CaptureExceptionMiddleware;
 
+//https://github.com/dotnet/aspnetcore/pull/47760
 public static class CaptureExceptionMiddlewareExtensions
 {
     public static IApplicationBuilder UseCaptureException(this IApplicationBuilder app)

diff --git a/src/Vertical.Slice.Template/Products/ProductConfigurations.cs b/src/Vertical.Slice.Template/Products/ProductConfigurations.cs
@@ -13,17 +13,17 @@ internal static class ProductConfigurations
     public const string Tag = "Products";
     public const string ProductsPrefixUri = $"{CatalogsConfigurations.CatalogsPrefixUri}/products";
 
-    public static WebApplicationBuilder AddProductsServices(this WebApplicationBuilder builder)
+    public static WebApplicationBuilder AddProductsModuleServices(this WebApplicationBuilder builder)
     {
         return builder;
     }
 
-    public static Task<WebApplication> UseProducts(this WebApplication app)
+    public static Task<WebApplication> UseProductsModule(this WebApplication app)
     {
         return Task.FromResult(app);
     }
 
-    public static IEndpointRouteBuilder MapProductsEndpoints(this IEndpointRouteBuilder endpoints)
+    public static IEndpointRouteBuilder MapProductsModuleEndpoints(this IEndpointRouteBuilder endpoints)
     {
         var products = endpoints.NewVersionedApi(Tag);
         var productsV1 = products.MapGroup(ProductsPrefixUri).HasDeprecatedApiVersion(0.9).HasApiVersion(1.0);

diff --git a/src/Vertical.Slice.Template/Shared/CatalogsConfigurations.cs b/src/Vertical.Slice.Template/Shared/CatalogsConfigurations.cs
@@ -14,11 +14,15 @@ public static class CatalogsConfigurations
     public static WebApplicationBuilder AddCatalogsServices(this WebApplicationBuilder builder)
     {
         // Shared
+        // Infrastructure
         builder.AddInfrastructures();
+
+        // Shared
+        // Catalogs Configurations
         builder.AddStorage();
 
         // Modules
-        builder.AddProductsServices();
+        builder.AddProductsModuleServices();
 
         return builder;
     }
@@ -29,7 +33,7 @@ public static async Task<WebApplication> UseCatalogs(this WebApplication app)
         await app.UseInfrastructure();
 
         // Modules
-        await app.UseProducts();
+        await app.UseProductsModule();
 
         return app;
     }
@@ -40,7 +44,7 @@ public static IEndpointRouteBuilder MapCatalogsEndpoints(this IEndpointRouteBuil
         endpoints.MapGet("/", () => "Catalogs  Api.").ExcludeFromDescription();
 
         // Modules
-        endpoints.MapProductsEndpoints();
+        endpoints.MapProductsModuleEndpoints();
 
         return endpoints;
     }

diff --git a/...ate/Shared/Extensions/WebApplicationExtensions/WebApplicationExtensions.Infrastructure.cs b/...ate/Shared/Extensions/WebApplicationExtensions/WebApplicationExtensions.Infrastructure.cs
@@ -25,6 +25,7 @@ public static Task UseInfrastructure(this WebApplication app)
             app.UseDeveloperExceptionPage();
 
             // https://github.com/dotnet/aspnetcore/issues/4765
+            // https://github.com/dotnet/aspnetcore/pull/47760
             app.UseCaptureException();
         }