[AppInsights] changing initializer to not use Add()

Azure · Jun 4, 2019 · 44af3a1 · 44af3a1
1 parent ad02f1c
commit 44af3a1
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/WebJobs.Script/Config/ScriptTelemetryInitializer.cs b/src/WebJobs.Script/Config/ScriptTelemetryInitializer.cs
@@ -2,6 +2,7 @@
 // Licensed under the MIT License. See License.txt in the project root for license information.
 
 using System;
+using System.Collections.Generic;
 using Microsoft.ApplicationInsights.Channel;
 using Microsoft.ApplicationInsights.Extensibility;
 using Microsoft.Extensions.Options;
@@ -29,7 +30,14 @@ public ScriptTelemetryInitializer(IOptions<ScriptJobHostOptions> hostOptions)
 
         public void Initialize(ITelemetry telemetry)
         {
-            telemetry?.Context?.Properties?.Add(ScriptConstants.LogPropertyHostInstanceIdKey, _hostOptions.InstanceId);
+            IDictionary<string, string> telemetryProps = telemetry?.Context?.Properties;
+
+            if (telemetryProps == null)
+            {
+                return;
+            }
+
+            telemetryProps[ScriptConstants.LogPropertyHostInstanceIdKey] = _hostOptions.InstanceId;
         }
     }
 }