Only add ExecuteRequestStep in integrated piepline mode and swallow any exceptions

Author: Liudmila Molkova

src/Microsoft.AspNet.TelemetryCorrelation/AspNetTelemetryCorrelationEventSource.cs

@@ -64,6 +64,12 @@ public void ActivityRestored(string id)
         {
             WriteEvent(8, id);
         }
+
+        [Event(9, Message = "Failed to invoke OnExecuteRequestStep, Error='{0}'", Level = EventLevel.Error)]
+        public void OnExecuteRequestStepInvokationError(string error)
+        {
+            WriteEvent(9, error);
+        }
     }
 }
 #pragma warning restore SA1600 // Elements must be documented

src/Microsoft.AspNet.TelemetryCorrelation/TelemetryCorrelationHttpModule.cs

@@ -35,9 +35,16 @@ public void Init(HttpApplication context)
             // OnExecuteRequestStep is availabile starting with 4.7.1
             // If this is executed in 4.7.1 runtime (regardless of targeted .NET version),
             // we will use it to restore lost activity, otherwise keep PreRequestHandlerExecute
-            if (onStepMethodInfo != null)
+            if (onStepMethodInfo != null && HttpRuntime.UsingIntegratedPipeline)
             {
-                onStepMethodInfo.Invoke(context, new object[] { (Action<HttpContextBase, Action>)OnExecuteRequestStep });
+                try
+                {
+                    onStepMethodInfo.Invoke(context, new object[] { (Action<HttpContextBase, Action>)OnExecuteRequestStep });
+                }
+                catch (Exception e)
+                {
+                    AspNetTelemetryCorrelationEventSource.Log.OnExecuteRequestStepInvokationError(e.Message);
+                }
             }
             else
             {

tools/Microsoft.AspNet.TelemetryCorrelation.settings.targets

@@ -6,7 +6,7 @@
         <VersionStartYear>2018</VersionStartYear>
         <VersionMajor>1</VersionMajor>
         <VersionMinor>0</VersionMinor>
-        <VersionRelease>2</VersionRelease>
+        <VersionRelease>3</VersionRelease>
         <VersionRelease Condition="'$(BuildQuality)' != 'rtm'">$(VersionRelease)-$(BuildQuality)</VersionRelease>
     </PropertyGroup>