Use default value for RabbitMQActivitySource.UseRoutingKeyAsOperationName

lukebakken · lukebakken · commit e54375d67100 · 2024-06-06T11:02:45.000-07:00
The default is `true`, anyway. No need to set it here. Pointed out by @kielex in this comment: #1533 (comment)
diff --git a/projects/RabbitMQ.Client.OpenTelemetry/TraceProviderBuilderExtensions.cs b/projects/RabbitMQ.Client.OpenTelemetry/TraceProviderBuilderExtensions.cs
@@ -12,7 +12,6 @@ public static class OpenTelemetryExtensions
     {
         public static TracerProviderBuilder AddRabbitMQInstrumentation(this TracerProviderBuilder builder)
         {
-            RabbitMQActivitySource.UseRoutingKeyAsOperationName = true;
             RabbitMQActivitySource.ContextExtractor = OpenTelemetryContextExtractor;
             RabbitMQActivitySource.ContextInjector = OpenTelemetryContextInjector;
             builder.AddSource("RabbitMQ.Client.*");

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,6 @@ public static class OpenTelemetryExtensions`
`12`	`12`	`{`
`13`	`13`	`public static TracerProviderBuilder AddRabbitMQInstrumentation(this TracerProviderBuilder builder)`
`14`	`14`	`{`
`15`		`- RabbitMQActivitySource.UseRoutingKeyAsOperationName = true;`
`16`	`15`	`RabbitMQActivitySource.ContextExtractor = OpenTelemetryContextExtractor;`
`17`	`16`	`RabbitMQActivitySource.ContextInjector = OpenTelemetryContextInjector;`
`18`	`17`	`builder.AddSource("RabbitMQ.Client.*");`