Support performance

samples/Sentry.Samples.AspNetCore.Basic/Program.cs

@@ -2,6 +2,7 @@
 using Microsoft.AspNetCore;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Hosting;
+using Microsoft.AspNetCore.Routing;
 using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.DependencyInjection;
 
@@ -26,20 +27,24 @@ public static IWebHost BuildWebHost(string[] args) =>
                 // .UseSentry("dsn") or .UseSentry(o => o.Dsn = ""; o.Release = "1.0"; ...)
 
                 // The App:
-                .Configure(a =>
+                .Configure(app =>
                 {
                     // An example ASP.NET Core middleware that throws an
                     // exception when serving a request to path: /throw
-                    a.Use(async (context, next) =>
+                    app.UseRouting();
+                    app.UseEndpoints(endpoints =>
                     {
-                        var log = context.RequestServices.GetService<ILoggerFactory>()
-                            .CreateLogger<Program>();
+                        // Reported events will be grouped by route pattern
+                        endpoints.MapGet("/throw/{message?}", context =>
+                        {
+                            var exceptionMessage = context.GetRouteValue("message") as string;
 
-                        log.LogInformation("Handling some request...");
+                            var log = context.RequestServices.GetRequiredService<ILoggerFactory>()
+                                .CreateLogger<Program>();
 
-                        if (context.Request.Path == "/throw")
-                        {
-                            var hub = context.RequestServices.GetService<IHub>();
+                            log.LogInformation("Handling some request...");
+
+                            var hub = context.RequestServices.GetRequiredService<IHub>();
                             hub.ConfigureScope(s =>
                             {
                                 // More data can be added to the scope like this:
@@ -53,10 +58,10 @@ public static IWebHost BuildWebHost(string[] args) =>
                             // The following exception will be captured by the SDK and the event
                             // will include the Log messages and any custom scope modifications
                             // as exemplified above.
-                            throw new Exception("An exception thrown from the ASP.NET Core pipeline");
-                        }
-
-                        await next();
+                            throw new Exception(
+                                exceptionMessage ?? "An exception thrown from the ASP.NET Core pipeline"
+                            );
+                        });
                     });
                 })
                 .Build();

samples/Sentry.Samples.AspNetCore.Basic/Properties/launchSettings.json

@@ -3,7 +3,7 @@
     "windowsAuthentication": false,
     "anonymousAuthentication": true,
     "iisExpress": {
-      "applicationUrl": "http://localhost:59739/",
+      "applicationUrl": "http://localhost:59739",
       "sslPort": 0
     }
   },
@@ -27,7 +27,7 @@
         "SENTRY_RELEASE": "e386dfd",
         "ASPNETCORE_ENVIRONMENT": "Development"
       },
-      "applicationUrl": "http://localhost:59740/"
+      "applicationUrl": "http://localhost:59740"
     }
   }
 }

samples/Sentry.Samples.AspNetCore.Basic/Sentry.Samples.AspNetCore.Basic.csproj

@@ -4,10 +4,6 @@
     <TargetFramework>net5.0</TargetFramework>
   </PropertyGroup>
 
-  <ItemGroup>
-    <PackageReference Include="Microsoft.AspNetCore" Version="2.1.2" />
-  </ItemGroup>
-
   <ItemGroup>
     <ProjectReference Include="..\..\src\Sentry.AspNetCore\Sentry.AspNetCore.csproj" />
   </ItemGroup>

src/Sentry.AspNetCore/Extensions/HttpContextExtensions.cs

@@ -0,0 +1,43 @@
+using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Routing;
+
+#if !NETSTANDARD2_0
+using Microsoft.AspNetCore.Http.Features;
+#endif
+
+namespace Sentry.AspNetCore.Extensions
+{
+    internal static class HttpContextExtensions
+    {
+        public static string? TryGetRouteTemplate(this HttpContext context)
+        {
+#if !NETSTANDARD2_0
+            // Requires .UseRouting()/.UseEndpoints()
+            var endpoint = context.Features.Get<IEndpointFeature?>()?.Endpoint as RouteEndpoint;
+            var routePattern = endpoint?.RoutePattern.RawText;
+
+            if (!string.IsNullOrWhiteSpace(routePattern))
+            {
+                return routePattern;
+            }
+#endif
+
+            // Requires legacy .UseMvc()
+            var routeData = context.GetRouteData();
+            var controller = routeData.Values["controller"]?.ToString();
+            var action = routeData.Values["action"]?.ToString();
+            var area = routeData.Values["area"]?.ToString();
+
+            if (!string.IsNullOrWhiteSpace(action))
+            {
+                return !string.IsNullOrWhiteSpace(area)
+                    ? $"{controller}.{action}"
+                    : $"{area}.{controller}.{action}";
+            }
+
+            // If the handler doesn't use routing (i.e. it checks `context.Request.Path` directly),
+            // then there is no way for us to extract anything that resembles a route template.
+            return null;
+        }
+    }
+}

src/Sentry.AspNetCore/ScopeExtensions.cs

@@ -7,6 +7,7 @@
 using Microsoft.AspNetCore.Routing;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Net.Http.Headers;
+using Sentry.AspNetCore.Extensions;
 using Sentry.Extensibility;
 using Sentry.Protocol;
 
@@ -68,29 +69,26 @@ public static void Populate(this Scope scope, HttpContext context, SentryAspNetC
             try
             {
                 var routeData = context.GetRouteData();
-                if (routeData != null)
+                var controller = routeData.Values["controller"]?.ToString();
+                var action = routeData.Values["action"]?.ToString();
+                var area = routeData.Values["area"]?.ToString();
+
+                if (controller != null)
+                {
+                    scope.SetTag("route.controller", controller);
+                }
+
+                if (action != null)
                 {
-                    var controller = routeData.Values["controller"]?.ToString();
-                    var action = routeData.Values["action"]?.ToString();
-                    var area = routeData.Values["area"]?.ToString();
-
-                    if (controller != null)
-                    {
-                        scope.SetTag("route.controller", controller);
-                    }
-                    if (action != null)
-                    {
-                        scope.SetTag("route.action", action);
-                    }
-                    if (area != null)
-                    {
-                        scope.SetTag("route.area", area);
-                    }
-
-                    scope.TransactionName = area == null
-                        ? $"{controller}.{action}"
-                        : $"{area}.{controller}.{action}";
+                    scope.SetTag("route.action", action);
                 }
+
+                if (area != null)
+                {
+                    scope.SetTag("route.area", area);
+                }
+
+                scope.TransactionName = context.TryGetRouteTemplate() ?? context.Request.Path;
             }
             catch(Exception e)
             {

src/Sentry.AspNetCore/SentryMiddleware.cs

@@ -12,6 +12,7 @@
 using Microsoft.AspNetCore.Routing;
 using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Options;
+using Sentry.AspNetCore.Extensions;
 using Sentry.Extensibility;
 using Sentry.Protocol;
 using Sentry.Reflection;
@@ -96,17 +97,6 @@ public async Task InvokeAsync(HttpContext context)
                     });
                 }
 
-                var routeData = context.GetRouteData();
-                var controller = routeData.Values["controller"]?.ToString();
-                var action = routeData.Values["action"]?.ToString();
-                var area = routeData.Values["area"]?.ToString();
-
-                // TODO: What if it's not using controllers (i.e. endpoints)?
-
-                var transactionName = area == null
-                    ? $"{controller}.{action}"
-                    : $"{area}.{controller}.{action}";
-
                 hub.ConfigureScope(scope =>
                 {
                     // At the point lots of stuff from the request are not yet filled
@@ -119,14 +109,18 @@ public async Task InvokeAsync(HttpContext context)
                     scope.OnEvaluating += (_, __) => PopulateScope(context, scope);
                 });
 
-                var transaction = hub.CreateTransaction(transactionName, "http.server");
+                var transaction = hub.CreateTransaction(
+                    // Try to get the route template or fallback to the request path
+                    context.TryGetRouteTemplate() ?? context.Request.Path,
+                    "http.server"
+                );
 
                 try
                 {
                     await _next(context).ConfigureAwait(false);
 
                     // When an exception was handled by other component (i.e: UseExceptionHandler feature).
-                    var exceptionFeature = context.Features.Get<IExceptionHandlerFeature>();
+                    var exceptionFeature = context.Features.Get<IExceptionHandlerFeature?>();
                     if (exceptionFeature?.Error != null)
                     {
                         CaptureException(exceptionFeature.Error);