Skip to content

Commit cb16901

Browse files
Adding a null check for routingFeature. (#10933)
* Adding a null check for routingFeature.
1 parent d1e06f1 commit cb16901

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

src/WebJobs.Script/Diagnostics/OpenTelemetry/OpenTelemetryConfigurationExtensions.cs

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -94,14 +94,19 @@ private static IOpenTelemetryBuilder ConfigureTracing(this IOpenTelemetryBuilder
9494
{
9595
o.EnrichWithHttpResponse = (activity, httpResponse) =>
9696
{
97-
if (Activity.Current != null)
97+
if (Activity.Current is not null)
9898
{
99-
var routingFeature = httpResponse.HttpContext.Features.Get<AspNetCore.Routing.IRoutingFeature>();
100-
var template = routingFeature.RouteData.Routers.FirstOrDefault(r => r is Route) as Route;
99+
Activity.Current.AddTag(ResourceSemanticConventions.FaaSTrigger, OpenTelemetryConstants.HttpTriggerType);
100+
101+
var routingFeature = httpResponse.HttpContext.Features.Get<IRoutingFeature>();
102+
if (routingFeature is null)
103+
{
104+
return;
105+
}
101106

107+
var template = routingFeature.RouteData.Routers.FirstOrDefault(r => r is Route) as Route;
102108
Activity.Current.DisplayName = $"{Activity.Current.DisplayName} {template?.RouteTemplate}";
103109
Activity.Current.AddTag(ResourceSemanticConventions.HttpRoute, template?.RouteTemplate);
104-
Activity.Current.AddTag(ResourceSemanticConventions.FaaSTrigger, OpenTelemetryConstants.HttpTriggerType);
105110
}
106111
};
107112
});

0 commit comments

Comments
 (0)