Do not read claims from SqlRoleProvider under classic ASP.NET (#2377)

* Do not read claims from SqlMembershipProvider under classic ASP.NET * check for role providers only
elastic · Jun 17, 2024 · 3cbf599 · 3cbf599
1 parent fccf2df
commit 3cbf599
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/integrations/Elastic.Apm.AspNetFullFramework/Elastic.Apm.AspNetFullFramework.csproj b/src/integrations/Elastic.Apm.AspNetFullFramework/Elastic.Apm.AspNetFullFramework.csproj
@@ -18,6 +18,7 @@
     <Reference Include="System.Configuration" />
     <Reference Include="System.Core" />
     <Reference Include="System.Web" />
+    <Reference Include="System.Web.ApplicationServices" />
     <ProjectReference Include="..\..\Elastic.Apm\Elastic.Apm.csproj" />
   </ItemGroup>
 

diff --git a/src/integrations/Elastic.Apm.AspNetFullFramework/ElasticApmModule.cs b/src/integrations/Elastic.Apm.AspNetFullFramework/ElasticApmModule.cs
@@ -631,8 +631,8 @@ private void FillSampledTransactionContextUser(HttpContext context, ITransaction
 				return;
 
 			var user = new User { UserName = userIdentity.Name };
-
-			if (context.User is ClaimsPrincipal claimsPrincipal)
+			var sqlRoleProvider = System.Web.Security.Roles.Providers.Cast<object>().Any(provider => provider.GetType().Name == "SqlRoleProvider");
+			if (!sqlRoleProvider && context.User is ClaimsPrincipal claimsPrincipal)
 			{
 				try
 				{