Merge pull request #97036 from geann/patch-1

Jak-MS · web-flow · commit 39b6607dbcd7 · 2022-08-19T14:21:32.000-07:00
Code snippet improvement
diff --git a/articles/azure-monitor/app/asp-net-exceptions.md b/articles/azure-monitor/app/asp-net-exceptions.md
@@ -247,7 +247,8 @@ namespace MVC2App.Controllers
         {
             if (filterContext != null && filterContext.HttpContext != null && filterContext.Exception != null)
             {
-                //If customError is Off, then AI HTTPModule will report the exception
+                //The attribute should track exceptions only when CustomErrors setting is On
+                //if CustomErrors is Off, exceptions will be caught by AI HTTP Module
                 if (filterContext.HttpContext.IsCustomErrorEnabled)
                 {   //or reuse instance (recommended!). see note above
                     var ai = new TelemetryClient();

Original file line number	Diff line number	Diff line change
`@@ -247,7 +247,8 @@ namespace MVC2App.Controllers`
`247`	`247`	`{`
`248`	`248`	`if (filterContext != null && filterContext.HttpContext != null && filterContext.Exception != null)`
`249`	`249`	`{`
`250`		`- //If customError is Off, then AI HTTPModule will report the exception`
	`250`	`+ //The attribute should track exceptions only when CustomErrors setting is On`
	`251`	`+ //if CustomErrors is Off, exceptions will be caught by AI HTTP Module`
`251`	`252`	`if (filterContext.HttpContext.IsCustomErrorEnabled)`
`252`	`253`	`{ //or reuse instance (recommended!). see note above`
`253`	`254`	`var ai = new TelemetryClient();`