Skip to content

Commit

Permalink
Update test/OpenTelemetry.Instrumentation.AspNetCore.Tests/RouteTests…
Browse files Browse the repository at this point in the history
…/TestApplication/TestApplicationFactory.cs
  • Loading branch information
alanwest committed Jan 27, 2024
1 parent e9a836b commit e340c76
Showing 1 changed file with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -183,11 +183,11 @@ private static WebApplication CreateExceptionHandlerApplication()
app.Urls.Clear();
app.Urls.Add("http://[::1]:0");

// TODO: Remove this condition once ASP.NET Core 8.0.2.
// Currently, .NET 8 has a different behavior than .NET 6 and 7.
// This is because ASP.NET Core 8+ has native metric instrumentation.
// When ASP.NET Core 8.0.2 is released then its behavior will align with .NET 6/7.
// See: https://github.com/dotnet/aspnetcore/issues/52648#issuecomment-1853432776
// TODO: Remove this condition once ASP.NET Core 8.0.2.
// Currently, .NET 8 has a different behavior than .NET 6 and 7.
// This is because ASP.NET Core 8+ has native metric instrumentation.
// When ASP.NET Core 8.0.2 is released then its behavior will align with .NET 6/7.
// See: https://github.com/dotnet/aspnetcore/issues/52648#issuecomment-1853432776
#if !NET8_0_OR_GREATER
app.MapGet("/Exception", (ctx) => throw new ApplicationException());
#else
Expand Down

0 comments on commit e340c76

Please sign in to comment.