Log when we detect LegacyAspNetSynchronizationContext (#2136)

elastic · Jul 26, 2023 · 4dd9b04 · 4dd9b04
1 parent 69fa0f7
commit 4dd9b04
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/src/integrations/Elastic.Apm.AspNetFullFramework/ElasticApmModule.cs b/src/integrations/Elastic.Apm.AspNetFullFramework/ElasticApmModule.cs
@@ -8,6 +8,7 @@
 using System.Collections.Specialized;
 using System.Linq;
 using System.Security.Claims;
+using System.Threading;
 using System.Web;
 using Elastic.Apm.Api;
 using Elastic.Apm.AspNetFullFramework.Extensions;
@@ -118,6 +119,17 @@ private void OnBeginRequest(object sender, EventArgs e)
 		{
 			_logger.Debug()?.Log("Incoming request processing started - starting trace...");
 
+			try
+			{
+				var usingLegacySynchronizationContext = SynchronizationContext.Current?.GetType().Name == "LegacyAspNetSynchronizationContext";
+				if (usingLegacySynchronizationContext)
+					_logger.Warning()?.Log("ASP.NET is using LegacyAspNetSynchronizationContext and might not behave well for asynchronous code");
+			}
+			catch
+			{
+				// ignored
+			}
+
 			try
 			{
 				ProcessBeginRequest(sender);