[SqlClient] Remove non-functional netfx properties

martincostello · martincostello · commit 45ce4b32be5f · 2025-09-08T09:40:24.000+01:00
Remove members from the public API that no nothing on .NET Framework.
diff --git a/build/Common.prod.props b/build/Common.prod.props
@@ -86,6 +86,7 @@
       <!-- Note: This resolves all the PublicApiAnalyzers files which are actually used by the analyzer -->
       <AdditionalFiles Include=".publicApi\PublicAPI.*.txt" />
       <AdditionalFiles Include=".publicApi\$(TargetFramework)\PublicAPI.*.txt" />
+      <AdditionalFiles Include=".publicApi\$([MSBuild]::GetTargetFrameworkIdentifier('$(TargetFramework)'))\PublicAPI.*.txt" />
     </ItemGroup>
   </Target>
 
diff --git a/src/OpenTelemetry.Instrumentation.SqlClient/.publicApi/.NETCoreApp/PublicAPI.Shipped.txt b/src/OpenTelemetry.Instrumentation.SqlClient/.publicApi/.NETCoreApp/PublicAPI.Shipped.txt
diff --git a/src/OpenTelemetry.Instrumentation.SqlClient/.publicApi/.NETCoreApp/PublicAPI.Unshipped.txt b/src/OpenTelemetry.Instrumentation.SqlClient/.publicApi/.NETCoreApp/PublicAPI.Unshipped.txt
@@ -0,0 +1,7 @@
+#nullable enable
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Enrich.get -> System.Action<System.Diagnostics.Activity!, string!, object!>?
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Enrich.set -> void
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Filter.get -> System.Func<object!, bool>?
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Filter.set -> void
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.RecordException.get -> bool
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.RecordException.set -> void
diff --git a/src/OpenTelemetry.Instrumentation.SqlClient/.publicApi/.NETStandard/PublicAPI.Shipped.txt b/src/OpenTelemetry.Instrumentation.SqlClient/.publicApi/.NETStandard/PublicAPI.Shipped.txt
@@ -0,0 +1 @@
+#nullable enable
diff --git a/src/OpenTelemetry.Instrumentation.SqlClient/.publicApi/.NETStandard/PublicAPI.Unshipped.txt b/src/OpenTelemetry.Instrumentation.SqlClient/.publicApi/.NETStandard/PublicAPI.Unshipped.txt
@@ -0,0 +1,7 @@
+#nullable enable
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Enrich.get -> System.Action<System.Diagnostics.Activity!, string!, object!>?
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Enrich.set -> void
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Filter.get -> System.Func<object!, bool>?
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Filter.set -> void
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.RecordException.get -> bool
+OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.RecordException.set -> void
diff --git a/src/OpenTelemetry.Instrumentation.SqlClient/.publicApi/PublicAPI.Unshipped.txt b/src/OpenTelemetry.Instrumentation.SqlClient/.publicApi/PublicAPI.Unshipped.txt
@@ -1,11 +1,5 @@
 #nullable enable
 OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions
-OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Enrich.get -> System.Action<System.Diagnostics.Activity!, string!, object!>?
-OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Enrich.set -> void
-OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Filter.get -> System.Func<object!, bool>?
-OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.Filter.set -> void
-OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.RecordException.get -> bool
-OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.RecordException.set -> void
 OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.SetDbQueryParameters.get -> bool
 OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.SetDbQueryParameters.set -> void
 OpenTelemetry.Instrumentation.SqlClient.SqlClientTraceInstrumentationOptions.SqlClientTraceInstrumentationOptions() -> void
diff --git a/src/OpenTelemetry.Instrumentation.SqlClient/CHANGELOG.md b/src/OpenTelemetry.Instrumentation.SqlClient/CHANGELOG.md
@@ -14,6 +14,10 @@
   Behaviors related to this option are now always enabled.
   ([#3072](https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/3072))
 
+* **Breaking change**: The `Enrich`, `Filter` and `RecordException` properties have
+  been removed for .NET Framework where they were non-functional.
+  ([#TODO](https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/TODO))
+
 ## 1.12.0-beta.2
 
 Released 2025-Jul-15
diff --git a/src/OpenTelemetry.Instrumentation.SqlClient/Implementation/SqlClientDiagnosticListener.cs b/src/OpenTelemetry.Instrumentation.SqlClient/Implementation/SqlClientDiagnosticListener.cs
@@ -127,13 +127,15 @@ public override void OnEventWritten(string name, object? payload)
                     {
                         try
                         {
+#if !NETFRAMEWORK
                             if (options.Filter?.Invoke(command) == false)
                             {
                                 SqlClientInstrumentationEventSource.Log.CommandIsFilteredOut(activity.OperationName);
                                 activity.IsAllDataRequested = false;
                                 activity.ActivityTraceFlags &= ~ActivityTraceFlags.Recorded;
                                 return;
                             }
+#endif
                         }
                         catch (Exception ex)
                         {
@@ -175,6 +177,7 @@ public override void OnEventWritten(string name, object? payload)
                             }
                         }
 
+#if !NETFRAMEWORK
                         try
                         {
                             options.Enrich?.Invoke(activity, "OnCustom", command);
@@ -183,6 +186,7 @@ public override void OnEventWritten(string name, object? payload)
                         {
                             SqlClientInstrumentationEventSource.Log.EnrichmentException(ex);
                         }
+#endif
                     }
                 }
 
@@ -261,10 +265,12 @@ public override void OnEventWritten(string name, object? payload)
 
                                 activity.SetStatus(ActivityStatusCode.Error, exception.Message);
 
+#if !NETFRAMEWORK
                                 if (options.RecordException)
                                 {
                                     activity.AddException(exception);
                                 }
+#endif
                             }
                             else
                             {
diff --git a/src/OpenTelemetry.Instrumentation.SqlClient/SqlClientTraceInstrumentationOptions.cs b/src/OpenTelemetry.Instrumentation.SqlClient/SqlClientTraceInstrumentationOptions.cs
@@ -1,7 +1,9 @@
 // Copyright The OpenTelemetry Authors
 // SPDX-License-Identifier: Apache-2.0
 
+#if !NETFRAMEWORK
 using System.Diagnostics;
+#endif
 using Microsoft.Extensions.Configuration;
 #if NET
 using OpenTelemetry.Instrumentation.SqlClient.Implementation;
@@ -47,6 +49,7 @@ internal SqlClientTraceInstrumentationOptions(IConfiguration configuration)
 #endif
     }
 
+#if !NETFRAMEWORK
     /// <summary>
     /// Gets or sets an action to enrich an <see cref="Activity"/> with the
     /// raw <c>SqlCommand</c> object.
@@ -97,6 +100,7 @@ internal SqlClientTraceInstrumentationOptions(IConfiguration configuration)
     /// href="https://github.com/open-telemetry/semantic-conventions/blob/main/docs/exceptions/exceptions-spans.md"/>.</para>
     /// </remarks>
     public bool RecordException { get; set; }
+#endif
 
     /// <summary>
     /// Gets or sets a value indicating whether or not the <see cref="SqlClientInstrumentation"/>
diff --git a/test/OpenTelemetry.Instrumentation.SqlClient.Tests/SqlClientIntegrationTests.cs b/test/OpenTelemetry.Instrumentation.SqlClient.Tests/SqlClientIntegrationTests.cs
@@ -57,7 +57,12 @@ public void SuccessfulCommandTest(
         using var tracerProvider = Sdk.CreateTracerProviderBuilder()
             .SetSampler(sampler)
             .AddInMemoryExporter(activities)
-            .AddSqlClientInstrumentation(options => options.RecordException = recordException)
+            .AddSqlClientInstrumentation(options =>
+            {
+#if NET
+                options.RecordException = recordException;
+#endif
+            })
             .Build();
 
         using var sqlConnection = new SqlConnection(this.GetConnectionString());
diff --git a/test/OpenTelemetry.Instrumentation.SqlClient.Tests/SqlClientTests.cs b/test/OpenTelemetry.Instrumentation.SqlClient.Tests/SqlClientTests.cs
@@ -141,7 +141,12 @@ public void SpansAndMetricsGeneratedOnlyWhenEnabled(bool tracesEnabled, bool met
         if (tracesEnabled)
         {
             tracerProviderBuilder
-                .AddSqlClientInstrumentation(options => options.RecordException = true)
+                .AddSqlClientInstrumentation(options =>
+                {
+#if NET
+                    options.RecordException = true;
+#endif
+                })
                 .AddInMemoryExporter(activities);
         }
 
@@ -337,7 +342,9 @@ private void RunSqlClientTestCase(
             .SetSampler(sampler)
             .AddSqlClientInstrumentation(options =>
             {
+#if NET
                 options.RecordException = true;
+#endif
                 options.EmitOldAttributes = emitOldAttributes;
                 options.EmitNewAttributes = emitNewAttributes;
             })
diff --git a/test/OpenTelemetry.Instrumentation.SqlClient.Tests/SqlClientTraceInstrumentationOptionsTests.cs b/test/OpenTelemetry.Instrumentation.SqlClient.Tests/SqlClientTraceInstrumentationOptionsTests.cs
@@ -233,6 +233,7 @@ private static void ActivityEnrichment(Activity activity, string method, object
         }
     }
 
+#if NET
     private Activity[] RunCommandWithFilter(
         string commandText,
         Func<object, bool> filter)
@@ -252,4 +253,5 @@ private Activity[] RunCommandWithFilter(
 
         return [.. activities];
     }
+#endif
 }

Original file line number	Diff line number	Diff line change
`@@ -127,13 +127,15 @@ public override void OnEventWritten(string name, object? payload)`
`127`	`127`	`{`
`128`	`128`	`try`
`129`	`129`	`{`
	`130`	`+#if !NETFRAMEWORK`
`130`	`131`	`if (options.Filter?.Invoke(command) == false)`
`131`	`132`	`{`
`132`	`133`	`SqlClientInstrumentationEventSource.Log.CommandIsFilteredOut(activity.OperationName);`
`133`	`134`	`activity.IsAllDataRequested = false;`
`134`	`135`	`activity.ActivityTraceFlags &= ~ActivityTraceFlags.Recorded;`
`135`	`136`	`return;`
`136`	`137`	`}`
	`138`	`+#endif`
`137`	`139`	`}`
`138`	`140`	`catch (Exception ex)`
`139`	`141`	`{`
`@@ -175,6 +177,7 @@ public override void OnEventWritten(string name, object? payload)`
`175`	`177`	`}`
`176`	`178`	`}`
`177`	`179`
	`180`	`+#if !NETFRAMEWORK`
`178`	`181`	`try`
`179`	`182`	`{`
`180`	`183`	`options.Enrich?.Invoke(activity, "OnCustom", command);`
`@@ -183,6 +186,7 @@ public override void OnEventWritten(string name, object? payload)`
`183`	`186`	`{`
`184`	`187`	`SqlClientInstrumentationEventSource.Log.EnrichmentException(ex);`
`185`	`188`	`}`
	`189`	`+#endif`
`186`	`190`	`}`
`187`	`191`	`}`
`188`	`192`
`@@ -261,10 +265,12 @@ public override void OnEventWritten(string name, object? payload)`
`261`	`265`
`262`	`266`	`activity.SetStatus(ActivityStatusCode.Error, exception.Message);`
`263`	`267`
	`268`	`+#if !NETFRAMEWORK`
`264`	`269`	`if (options.RecordException)`
`265`	`270`	`{`
`266`	`271`	`activity.AddException(exception);`
`267`	`272`	`}`
	`273`	`+#endif`
`268`	`274`	`}`
`269`	`275`	`else`
`270`	`276`	`{`