Support SkipStatusCodePages on endpoints and authorized routes (#38509)

* Support SkipStatusCodePages on endpoints and authorized routes

* Address feedback from API review

* Fix sort of usings

Author: captainsafia

src/Http/Http.Abstractions/src/Metadata/IStatusCodePagesMetadata.cs

@@ -0,0 +1,12 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+namespace Microsoft.AspNetCore.Http.Metadata;
+
+/// <summary>
+/// Defines a contract used to specify metadata for skipping the StatusCodePage
+/// middleware in <see cref="Endpoint.Metadata"/>.
+/// </summary>
+public interface ISkipStatusCodePagesMetadata
+{
+}

src/Http/Http.Abstractions/src/PublicAPI.Unshipped.txt

@@ -1,3 +1,4 @@
 #nullable enable
 *REMOVED*abstract Microsoft.AspNetCore.Http.HttpResponse.ContentType.get -> string!
 abstract Microsoft.AspNetCore.Http.HttpResponse.ContentType.get -> string?
+Microsoft.AspNetCore.Http.Metadata.ISkipStatusCodePagesMetadata

src/Middleware/Diagnostics/src/StatusCodePage/StatusCodePagesMiddleware.cs

@@ -5,6 +5,7 @@
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Http.Metadata;
 using Microsoft.Extensions.Options;
 
 namespace Microsoft.AspNetCore.Diagnostics;
@@ -41,6 +42,13 @@ public async Task Invoke(HttpContext context)
     {
         var statusCodeFeature = new StatusCodePagesFeature();
         context.Features.Set<IStatusCodePagesFeature>(statusCodeFeature);
+        var endpoint = context.GetEndpoint();
+        var skipStatusCodePageMetadata = endpoint?.Metadata.GetMetadata<ISkipStatusCodePagesMetadata>();
+
+        if (skipStatusCodePageMetadata is not null)
+        {
+            statusCodeFeature.Enabled = false;
+        }
 
         await _next(context);
 

src/Middleware/Diagnostics/test/UnitTests/Microsoft.AspNetCore.Diagnostics.Tests.csproj

@@ -20,6 +20,7 @@
     <Reference Include="Microsoft.Extensions.DiagnosticAdapter" />
     <Reference Include="Microsoft.Extensions.FileProviders.Embedded" />
     <Reference Include="Microsoft.AspNetCore" />
+    <Reference Include="Microsoft.AspNetCore.Mvc.ViewFeatures" />
   </ItemGroup>
 
 </Project>

src/Middleware/Diagnostics/test/UnitTests/StatusCodeMiddlewareTest.cs

@@ -8,6 +8,7 @@
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Hosting;
 using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Mvc;
 using Microsoft.AspNetCore.TestHost;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
@@ -283,4 +284,38 @@ public async Task Reexecute_WorksAfterUseRoutingWithGlobalRouteBuilder()
         var content = await response.Content.ReadAsStringAsync();
         Assert.Equal("errorPage", content);
     }
+
+    [Fact]
+    public async Task SkipStatusCodePages_SupportsEndpoints()
+    {
+        var builder = WebApplication.CreateBuilder();
+        builder.WebHost.UseTestServer();
+        await using var app = builder.Build();
+
+        app.UseRouting();
+
+        app.UseStatusCodePages();
+
+        app.UseEndpoints(endpoints =>
+        {
+            endpoints.MapGet("/", [SkipStatusCodePages] (c) =>
+            {
+                c.Response.StatusCode = 404;
+                return Task.CompletedTask;
+            });
+        });
+
+        app.Run((context) =>
+        {
+            throw new InvalidOperationException("Invalid input provided.");
+        });
+
+        await app.StartAsync();
+
+        using var server = app.GetTestServer();
+        var client = server.CreateClient();
+        var response = await client.GetAsync("/");
+        var content = await response.Content.ReadAsStringAsync();
+        Assert.Empty(content);
+    }
 }

src/Mvc/Mvc.ViewFeatures/src/SkipStatusCodePagesAttribute.cs

@@ -3,6 +3,7 @@
 
 using System;
 using Microsoft.AspNetCore.Diagnostics;
+using Microsoft.AspNetCore.Http.Metadata;
 using Microsoft.AspNetCore.Mvc.Filters;
 
 namespace Microsoft.AspNetCore.Mvc;
@@ -11,7 +12,7 @@ namespace Microsoft.AspNetCore.Mvc;
 /// A filter that prevents execution of the StatusCodePages middleware.
 /// </summary>
 [AttributeUsage(AttributeTargets.Class | AttributeTargets.Method, AllowMultiple = false, Inherited = true)]
-public class SkipStatusCodePagesAttribute : Attribute, IResourceFilter
+public class SkipStatusCodePagesAttribute : Attribute, IResourceFilter, ISkipStatusCodePagesMetadata
 {
     /// <inheritdoc />
     public void OnResourceExecuted(ResourceExecutedContext context)