Addressing warnings.

Lombiq · Jan 19, 2024 · ba9db63 · ba9db63
1 parent d165d1d
commit ba9db63
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/Lombiq.Hosting.Azure.ApplicationInsights/Services/NullTelemetryChannel.cs b/Lombiq.Hosting.Azure.ApplicationInsights/Services/NullTelemetryChannel.cs
@@ -11,9 +11,7 @@ internal sealed class NullTelemetryChannel : ITelemetryChannel
     public bool? DeveloperMode { get; set; }
     public string EndpointAddress { get; set; }
 
-#pragma warning disable S1186 // Methods should not be empty
     public void Dispose() { }
     public void Flush() { }
     public void Send(ITelemetry item) { }
-#pragma warning restore S1186 // Methods should not be empty
 }
diff --git a/...re.ApplicationInsights/TelemetryInitializers/UserContextPopulatingTelemetryInitializer.cs b/...re.ApplicationInsights/TelemetryInitializers/UserContextPopulatingTelemetryInitializer.cs
@@ -39,7 +39,7 @@ public void Initialize(ITelemetry telemetry)
         if (options.EnableUserAgentCollection && string.IsNullOrEmpty(requestTelemetry.Context.User.UserAgent))
         {
             // While there is requestTelemetry.Context.User.UserAgent that's not displayed on the Azure Portal.
-            requestTelemetry.TryAddProperty("UserAgent", httpContext.Request.Headers["User-Agent"]);
+            requestTelemetry.TryAddProperty("UserAgent", httpContext.Request.Headers.UserAgent);
         }
 
         if (options.EnableIpAddressCollection)