Use MapActionEndpointRouteBuilderExtensions for logger category

Author: halter73

src/Http/Routing/src/Internal/MapActionExpressionTreeBuilder.cs

@@ -11,6 +11,7 @@
 using System.Reflection;
 using System.Threading;
 using System.Threading.Tasks;
+using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Http.Metadata;
 using Microsoft.Extensions.DependencyInjection;
@@ -319,12 +320,7 @@ public static RequestDelegate BuildRequestDelegate(Delegate action)
         private static void LogRequestBodyIOException(HttpContext httpContext, IOException exception)
         {
             var loggerFactory = httpContext.RequestServices.GetRequiredService<ILoggerFactory>();
-
-            // REVIEW: I'm not sure how we feel about using an internal type as a category name, but this is what was done in other
-            // internal classes like DfaMather and DefaultLinkGenerator. My main concern is if we want to use the same category for
-            // other MapAction implementations (e.g. Roslyn source generators)
-            var logger = loggerFactory.CreateLogger(typeof(MapActionExpressionTreeBuilder).FullName!);
-
+            var logger = loggerFactory.CreateLogger(typeof(MapActionEndpointRouteBuilderExtensions).FullName!);
             Log.RequestBodyIOException(logger, exception);
         }
 

src/Http/Routing/test/UnitTests/Internal/MapActionExpressionTreeBuilderTest.cs

@@ -11,6 +11,7 @@
 using System.Text.Json;
 using System.Threading;
 using System.Threading.Tasks;
+using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Http.Features;
 using Microsoft.AspNetCore.Http.Metadata;
@@ -256,7 +257,7 @@ public async Task RequestDelegateLogsFromBodyIOExceptionsAsDebug()
         {
             var invoked = false;
 
-            var sink = new TestSink(context => context.LoggerName == typeof(MapActionExpressionTreeBuilder).FullName);
+            var sink = new TestSink(context => context.LoggerName == typeof(MapActionEndpointRouteBuilderExtensions).FullName);
             var testLoggerFactory = new TestLoggerFactory(sink, enabled: true);
 
             void TestAction([FromBody] Todo todo)
@@ -320,7 +321,7 @@ public async Task RequestDelegateLogsFromFormIOExceptionsAsDebug()
         {
             var invoked = false;
 
-            var sink = new TestSink(context => context.LoggerName == typeof(MapActionExpressionTreeBuilder).FullName);
+            var sink = new TestSink(context => context.LoggerName == typeof(MapActionEndpointRouteBuilderExtensions).FullName);
             var testLoggerFactory = new TestLoggerFactory(sink, enabled: true);
 
             void TestAction([FromForm] int value)