Enable Roslyn analysis - main project - final (#4707)

Kielek · web-flow · commit 43fec714e313 · 2025-12-15T06:32:08.000+01:00
* Enable analysis on whole src folder

* CA5392

* CA1310

* Mute CA1815

* CA1810

* Mute CA1065

* IDE0007

* Remove unused IntegrationRegistry and IntegrationInfo

* Mute CA1810

* Remove empty globalsuppresion file
diff --git a/OpenTelemetry.AutoInstrumentation.sln b/OpenTelemetry.AutoInstrumentation.sln
@@ -38,7 +38,6 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "src", "src", "{9E5F0022-0A5
 		src\CommonExcludedAssets.props = src\CommonExcludedAssets.props
 		src\Directory.Build.props = src\Directory.Build.props
 		src\Directory.Packages.props = src\Directory.Packages.props
-		src\GlobalSuppressions.cs = src\GlobalSuppressions.cs
 	EndProjectSection
 EndProject
 Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "OpenTelemetry.AutoInstrumentation.BuildTasks", "src\OpenTelemetry.AutoInstrumentation.BuildTasks\OpenTelemetry.AutoInstrumentation.BuildTasks.csproj", "{12A213FD-12A3-406C-8284-E7EA0B180428}"
diff --git a/src/Directory.Build.props b/src/Directory.Build.props
@@ -30,6 +30,7 @@
     <NoWarn>$(NoWarn);NU5104</NoWarn>
 
     <EmbedUntrackedSources Condition=" '$(TargetFramework)' == 'net462' ">true</EmbedUntrackedSources>
+    <AnalysisLevel>latest-All</AnalysisLevel>
   </PropertyGroup>
 
   <ItemGroup>
@@ -51,9 +52,6 @@
     <None Include="$(MSBuildThisFileDirectory)\..\opentelemetry-icon-color.png" Pack="true" Visible="false" PackagePath="opentelemetry-icon-color.png" />
     <None Include="docs\README.md" Pack="true" PackagePath="docs\README.md" />
 
-    <!-- StyleCop -->
-    <Compile Include="$(MSBuildThisFileDirectory)GlobalSuppressions.cs" Link="GlobalSuppressions.src.cs" />
-
     <!-- reference assemblies let us target .NET Framework without the SDK (for example, on non-Windows) -->
     <PackageReference Include="Microsoft.NETFramework.ReferenceAssemblies" PrivateAssets="all" />
   </ItemGroup>
diff --git a/src/GlobalSuppressions.cs b/src/GlobalSuppressions.cs
diff --git a/src/OpenTelemetry.AutoInstrumentation.AspNetCoreBootstrapper/OpenTelemetry.AutoInstrumentation.AspNetCoreBootstrapper.csproj b/src/OpenTelemetry.AutoInstrumentation.AspNetCoreBootstrapper/OpenTelemetry.AutoInstrumentation.AspNetCoreBootstrapper.csproj
@@ -5,7 +5,6 @@
   <PropertyGroup>
     <TargetFrameworks>net8.0</TargetFrameworks>
     <Description>ASP.NET Core Bootstrapper used by the OpenTelemetry.AutoInstrumentation project.</Description>
-    <AnalysisLevel>latest-All</AnalysisLevel>
   </PropertyGroup>
   
   <ItemGroup>
diff --git a/src/OpenTelemetry.AutoInstrumentation.BuildTasks/OpenTelemetry.AutoInstrumentation.BuildTasks.csproj b/src/OpenTelemetry.AutoInstrumentation.BuildTasks/OpenTelemetry.AutoInstrumentation.BuildTasks.csproj
@@ -13,7 +13,6 @@
 
     <!-- The version of certain packages is needed to copy dependencies when creating the NuGet package -->
     <NuGetVersioningVersion>6.5.0</NuGetVersioningVersion>
-    <AnalysisLevel>latest-All</AnalysisLevel>
   </PropertyGroup>
 
   <!-- ItemGroup with build files -->
diff --git a/src/OpenTelemetry.AutoInstrumentation.Loader/AssemblyResolver.Net.cs b/src/OpenTelemetry.AutoInstrumentation.Loader/AssemblyResolver.Net.cs
@@ -54,7 +54,7 @@ internal partial class AssemblyResolver
         }
         else
         {
-            var entry = StoreFiles?.FirstOrDefault(e => e.EndsWith($"{assemblyName.Name}.dll"));
+            var entry = StoreFiles?.FirstOrDefault(e => e.EndsWith($"{assemblyName.Name}.dll", StringComparison.Ordinal));
             if (entry != null)
             {
                 return DependencyLoadContext.LoadFromAssemblyPath(entry);
diff --git a/src/OpenTelemetry.AutoInstrumentation.Loader/AssemblyResolver.NetFramework.cs b/src/OpenTelemetry.AutoInstrumentation.Loader/AssemblyResolver.NetFramework.cs
@@ -100,6 +100,7 @@ internal partial class AssemblyResolver
     /// Return redirection table used in runtime that will match TFM folder to load assemblies.
     /// It may not be actual .NET Framework version.
     /// </summary>
+    [DefaultDllImportSearchPaths(DllImportSearchPath.SafeDirectories)]
     [DllImport("OpenTelemetry.AutoInstrumentation.Native.dll")]
     private static extern int GetNetFrameworkRedirectionVersion();
 
diff --git a/src/OpenTelemetry.AutoInstrumentation.Loader/OpenTelemetry.AutoInstrumentation.Loader.csproj b/src/OpenTelemetry.AutoInstrumentation.Loader/OpenTelemetry.AutoInstrumentation.Loader.csproj
@@ -5,6 +5,8 @@
     <TargetFrameworks Condition="'$(OS)' == 'Windows_NT'">$(TargetFrameworks);net462</TargetFrameworks>
     <Description>Loader used by the OpenTelemetry.AutoInstrumentation project.</Description>
     <OutputPath>..\bin\ProfilerResources\</OutputPath>
+    <!-- Do not catch general exception types. Typically, we need to handle all Exception to do not break application. -->
+    <NoWarn>$(NoWarn);CA1031</NoWarn>
   </PropertyGroup>
 <ItemGroup>
   <Compile Include="..\OpenTelemetry.AutoInstrumentation\Logging\InternalLogger.cs">
diff --git a/src/OpenTelemetry.AutoInstrumentation.StartupHook/OpenTelemetry.AutoInstrumentation.StartupHook.csproj b/src/OpenTelemetry.AutoInstrumentation.StartupHook/OpenTelemetry.AutoInstrumentation.StartupHook.csproj
@@ -8,7 +8,6 @@
     <Description>StartupHook used by the OpenTelemetry.AutoInstrumentation project.</Description>
     <!-- Suppress CA1031 warnings about catching general exception types. In this project we need to catch all exceptions to prevent the application from crashing. -->
     <NoWarn>$(NoWarn);CA1031</NoWarn>
-    <AnalysisLevel>latest-All</AnalysisLevel>
   </PropertyGroup>
 
   <ItemGroup>
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/CallTargetState.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/CallTargetState.cs
@@ -9,7 +9,9 @@ namespace OpenTelemetry.AutoInstrumentation.CallTarget;
 /// <summary>
 /// Call target execution state
 /// </summary>
+#pragma warning disable CA1815 // Override equals and operator equals on value types
 public readonly struct CallTargetState
+#pragma warning restore CA1815 // Override equals and operator equals on value types
 {
     private readonly Activity? _previousActivity;
     private readonly Activity? _activity;
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler.cs
@@ -12,7 +12,9 @@ internal static class BeginMethodHandler<TIntegration, TTarget>
 {
     private static readonly InvokeDelegate _invokeDelegate;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     static BeginMethodHandler()
+#pragma warning restore CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     {
         try
         {
@@ -24,7 +26,9 @@ static BeginMethodHandler()
         }
         catch (Exception ex)
         {
+#pragma warning disable CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
             throw new CallTargetInvokerException(ex);
+#pragma warning restore CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
         }
         finally
         {
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`1.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`1.cs
@@ -12,7 +12,9 @@ internal static class BeginMethodHandler<TIntegration, TTarget, TArg1>
 {
     private static readonly InvokeDelegate _invokeDelegate;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     static BeginMethodHandler()
+#pragma warning restore CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     {
         try
         {
@@ -25,7 +27,9 @@ static BeginMethodHandler()
         }
         catch (Exception ex)
         {
+#pragma warning disable CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
             throw new CallTargetInvokerException(ex);
+#pragma warning restore CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
         }
         finally
         {
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`2.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`2.cs
@@ -12,7 +12,9 @@ internal static class BeginMethodHandler<TIntegration, TTarget, TArg1, TArg2>
 {
     private static readonly InvokeDelegate _invokeDelegate;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     static BeginMethodHandler()
+#pragma warning restore CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     {
         try
         {
@@ -26,7 +28,9 @@ static BeginMethodHandler()
         }
         catch (Exception ex)
         {
+#pragma warning disable CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
             throw new CallTargetInvokerException(ex);
+#pragma warning restore CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
         }
         finally
         {
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`3.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`3.cs
@@ -12,7 +12,9 @@ internal static class BeginMethodHandler<TIntegration, TTarget, TArg1, TArg2, TA
 {
     private static readonly InvokeDelegate _invokeDelegate;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     static BeginMethodHandler()
+#pragma warning restore CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     {
         try
         {
@@ -27,7 +29,9 @@ static BeginMethodHandler()
         }
         catch (Exception ex)
         {
+#pragma warning disable CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
             throw new CallTargetInvokerException(ex);
+#pragma warning restore CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
         }
         finally
         {
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`4.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`4.cs
@@ -12,7 +12,9 @@ internal static class BeginMethodHandler<TIntegration, TTarget, TArg1, TArg2, TA
 {
     private static readonly InvokeDelegate _invokeDelegate;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     static BeginMethodHandler()
+#pragma warning restore CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     {
         try
         {
@@ -28,7 +30,9 @@ static BeginMethodHandler()
         }
         catch (Exception ex)
         {
+#pragma warning disable CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
             throw new CallTargetInvokerException(ex);
+#pragma warning restore CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
         }
         finally
         {
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`5.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`5.cs
@@ -12,7 +12,9 @@ internal static class BeginMethodHandler<TIntegration, TTarget, TArg1, TArg2, TA
 {
     private static readonly InvokeDelegate _invokeDelegate;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     static BeginMethodHandler()
+#pragma warning restore CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     {
         try
         {
@@ -29,7 +31,9 @@ static BeginMethodHandler()
         }
         catch (Exception ex)
         {
+#pragma warning disable CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
             throw new CallTargetInvokerException(ex);
+#pragma warning restore CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
         }
         finally
         {
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`6.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`6.cs
@@ -12,7 +12,9 @@ internal static class BeginMethodHandler<TIntegration, TTarget, TArg1, TArg2, TA
 {
     private static readonly InvokeDelegate _invokeDelegate;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     static BeginMethodHandler()
+#pragma warning restore CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     {
         try
         {
@@ -30,7 +32,9 @@ static BeginMethodHandler()
         }
         catch (Exception ex)
         {
+#pragma warning disable CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
             throw new CallTargetInvokerException(ex);
+#pragma warning restore CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
         }
         finally
         {
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`7.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`7.cs
@@ -12,7 +12,9 @@ internal static class BeginMethodHandler<TIntegration, TTarget, TArg1, TArg2, TA
 {
     private static readonly InvokeDelegate _invokeDelegate;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     static BeginMethodHandler()
+#pragma warning restore CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     {
         try
         {
@@ -31,7 +33,9 @@ static BeginMethodHandler()
         }
         catch (Exception ex)
         {
+#pragma warning disable CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
             throw new CallTargetInvokerException(ex);
+#pragma warning restore CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
         }
         finally
         {
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`8.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodHandler`8.cs
@@ -12,7 +12,9 @@ internal static class BeginMethodHandler<TIntegration, TTarget, TArg1, TArg2, TA
 {
     private static readonly InvokeDelegate _invokeDelegate;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     static BeginMethodHandler()
+#pragma warning restore CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     {
         try
         {
@@ -32,7 +34,9 @@ static BeginMethodHandler()
         }
         catch (Exception ex)
         {
+#pragma warning disable CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
             throw new CallTargetInvokerException(ex);
+#pragma warning restore CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
         }
         finally
         {
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodSlowHandler.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/BeginMethodSlowHandler.cs
@@ -10,7 +10,9 @@ internal static class BeginMethodSlowHandler<TIntegration, TTarget>
 {
     private static readonly InvokeDelegate _invokeDelegate;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     static BeginMethodSlowHandler()
+#pragma warning restore CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     {
         try
         {
@@ -22,7 +24,9 @@ static BeginMethodSlowHandler()
         }
         catch (Exception ex)
         {
+#pragma warning disable CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
             throw new CallTargetInvokerException(ex);
+#pragma warning restore CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
         }
         finally
         {
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/Continuations/TaskContinuationGenerator.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/Continuations/TaskContinuationGenerator.cs
@@ -10,7 +10,9 @@ internal class TaskContinuationGenerator<TIntegration, TTarget, TReturn> : Conti
     private static readonly ContinuationMethodDelegate? _continuation;
     private static readonly bool _preserveContext;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline
     static TaskContinuationGenerator()
+#pragma warning restore CA1810 // Initialize reference type static fields inline
     {
         var result = IntegrationMapper.CreateAsyncEndMethodDelegate(typeof(TIntegration), typeof(TTarget), typeof(object));
         if (result.Method != null)
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/Continuations/TaskContinuationGenerator`1.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/Continuations/TaskContinuationGenerator`1.cs
@@ -12,7 +12,9 @@ internal class TaskContinuationGenerator<TIntegration, TTarget, TReturn, TResult
     private static readonly ContinuationMethodDelegate? _continuation;
     private static readonly bool _preserveContext;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline
     static TaskContinuationGenerator()
+#pragma warning restore CA1810 // Initialize reference type static fields inline
     {
         var result = IntegrationMapper.CreateAsyncEndMethodDelegate(typeof(TIntegration), typeof(TTarget), typeof(TResult));
         if (result.Method != null)
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/Continuations/ValueTaskContinuationGenerator.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/Continuations/ValueTaskContinuationGenerator.cs
@@ -9,7 +9,9 @@ internal class ValueTaskContinuationGenerator<TIntegration, TTarget, TReturn> :
     private static readonly ContinuationMethodDelegate? _continuation;
     private static readonly bool _preserveContext;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline
     static ValueTaskContinuationGenerator()
+#pragma warning restore CA1810 // Initialize reference type static fields inline
     {
         var result = IntegrationMapper.CreateAsyncEndMethodDelegate(typeof(TIntegration), typeof(TTarget), typeof(object));
         if (result.Method != null)
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/Continuations/ValueTaskContinuationGenerator`1.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/Continuations/ValueTaskContinuationGenerator`1.cs
@@ -11,7 +11,9 @@ internal class ValueTaskContinuationGenerator<TIntegration, TTarget, TReturn, TR
     private static readonly ContinuationMethodDelegate? _continuation;
     private static readonly bool _preserveContext;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline
     static ValueTaskContinuationGenerator()
+#pragma warning restore CA1810 // Initialize reference type static fields inline
     {
         var result = IntegrationMapper.CreateAsyncEndMethodDelegate(typeof(TIntegration), typeof(TTarget), typeof(TResult));
         if (result.Method != null)
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/EndMethodHandler.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/EndMethodHandler.cs
@@ -9,7 +9,9 @@ internal static class EndMethodHandler<TIntegration, TTarget>
 {
     private static readonly InvokeDelegate _invokeDelegate;
 
+#pragma warning disable CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     static EndMethodHandler()
+#pragma warning restore CA1810 // Initialize reference type static fields inline. This static constructor is necessary for initializing the instrumentation delegate for bytecode instrumentation. Not possible to omit constructor due to exception trow scenario.
     {
         try
         {
@@ -21,7 +23,9 @@ static EndMethodHandler()
         }
         catch (Exception ex)
         {
+#pragma warning disable CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
             throw new CallTargetInvokerException(ex);
+#pragma warning restore CA1065 // Do not raise exceptions in unexpected locations. Needed for bytecode instrumentation.
         }
         finally
         {
diff --git a/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/EndMethodHandler`1.cs b/src/OpenTelemetry.AutoInstrumentation/CallTarget/Handlers/EndMethodHandler`1.cs
diff --git a/src/OpenTelemetry.AutoInstrumentation/Configurations/IntegrationInfo.cs b/src/OpenTelemetry.AutoInstrumentation/Configurations/IntegrationInfo.cs
diff --git a/src/OpenTelemetry.AutoInstrumentation/Configurations/IntegrationRegistry.cs b/src/OpenTelemetry.AutoInstrumentation/Configurations/IntegrationRegistry.cs
diff --git a/src/OpenTelemetry.AutoInstrumentation/DuckTyping/DuckType.Statics.cs b/src/OpenTelemetry.AutoInstrumentation/DuckTyping/DuckType.Statics.cs
diff --git a/src/OpenTelemetry.AutoInstrumentation/Instrumentations/Kafka/Integrations/MessageHeadersHelper.cs b/src/OpenTelemetry.AutoInstrumentation/Instrumentations/Kafka/Integrations/MessageHeadersHelper.cs
diff --git a/src/OpenTelemetry.AutoInstrumentation/Instrumentations/Kafka/Integrations/ProducerDeliveryHandlerActionIntegration.cs b/src/OpenTelemetry.AutoInstrumentation/Instrumentations/Kafka/Integrations/ProducerDeliveryHandlerActionIntegration.cs
diff --git a/src/OpenTelemetry.AutoInstrumentation/Instrumentations/Log4Net/Bridge/OpenTelemetryAppenderInitializer.cs b/src/OpenTelemetry.AutoInstrumentation/Instrumentations/Log4Net/Bridge/OpenTelemetryAppenderInitializer.cs
diff --git a/src/SourceGenerators/SourceGenerators.csproj b/src/SourceGenerators/SourceGenerators.csproj

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ internal partial class AssemblyResolver`
`54`	`54`	`}`
`55`	`55`	`else`
`56`	`56`	`{`
`57`		`- var entry = StoreFiles?.FirstOrDefault(e => e.EndsWith($"{assemblyName.Name}.dll"));`
	`57`	`+ var entry = StoreFiles?.FirstOrDefault(e => e.EndsWith($"{assemblyName.Name}.dll", StringComparison.Ordinal));`
`58`	`58`	`if (entry != null)`
`59`	`59`	`{`
`60`	`60`	`return DependencyLoadContext.LoadFromAssemblyPath(entry);`