Fixed runtime error when registering ApplicationInsights. Closes #53

jezzsantos · Oct 16, 2024 · 8753c6f · 8753c6f
1 parent ebe12b0
commit 8753c6f
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/Infrastructure.Web.Hosting.Common/Extensions/HostExtensions.cs b/src/Infrastructure.Web.Hosting.Common/Extensions/HostExtensions.cs
@@ -71,6 +71,11 @@ public static WebApplication ConfigureApiHost(this WebApplicationBuilder appBuil
         WebHostOptions hostOptions)
     {
         var services = appBuilder.Services;
+#if HOSTEDONAZURE
+        // HACK: We need to add this dependency before registering any keyed dependencies, for ApplicationInsights v2.22.0.
+        // See https://github.com/microsoft/ApplicationInsights-dotnet/issues/2879
+        services.AddApplicationInsightsTelemetry();
+#endif
         RegisterSharedServices();
         RegisterConfiguration(hostOptions.IsMultiTenanted);
         RegisterRecording();
@@ -171,8 +176,6 @@ void RegisterRecording()
 #else
 #if HOSTEDONAZURE
                 loggingBuilder.AddApplicationInsights();
-
-                services.AddApplicationInsightsTelemetry();
 #elif HOSTEDONAWS
                 loggingBuilder.AddLambdaLogger();
 #endif