Merge pull request #3 from SonicGD/fix-metric-current-route

Fixes #2. Check if curent route metric name was already added to context
AppMetrics · Nov 2, 2017 · 644440c · 644440c
2 parents 8ac415a + f216e78
commit 644440c
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/App.Metrics.AspNetCore.Core/Internal/Extensions/MetricsHttpContextExtensions.cs b/src/App.Metrics.AspNetCore.Core/Internal/Extensions/MetricsHttpContextExtensions.cs
@@ -14,7 +14,10 @@ internal static class MetricsHttpContextExtensions
 
         public static void AddMetricsCurrentRouteName(this HttpContext context, string metricName)
         {
-            context.Items.Add(MetricsCurrentRouteName, metricName);
+            if (!context.Items.ContainsKey(MetricsCurrentRouteName))
+            {
+                context.Items.Add(MetricsCurrentRouteName, metricName);
+            }
         }
 
         public static string GetMetricsCurrentRouteName(this HttpContext context)