Skip to content

Commit 059c893

Browse files
committed
Refactor: Move obsolete message to private constant to avoid repetition
1 parent 360fa59 commit 059c893

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/OpenTelemetry.Extensions.Propagators/JaegerPropagator.cs

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ namespace OpenTelemetry.Extensions.Propagators;
1010
/// <summary>
1111
/// A text map propagator for Jaeger trace context. See https://www.jaegertracing.io/docs/next-release/client-libraries/#propagation-format.
1212
/// </summary>
13-
[Obsolete("The Jaeger propagator is obsolete and will be removed in a future version. The Jaeger propagation format has been deprecated in favor of W3C Trace Context. Use TraceContextPropagator instead. See https://www.jaegertracing.io/sdk-migration/#propagation-format and https://github.com/open-telemetry/opentelemetry-specification/issues/4827 for more information.")]
13+
[Obsolete(ObsoleteMessage)]
1414
public class JaegerPropagator : TextMapPropagator
1515
{
1616
internal const string JaegerHeader = "uber-trace-id";
@@ -20,6 +20,8 @@ public class JaegerPropagator : TextMapPropagator
2020

2121
internal static readonly string[] JaegerDelimiters = [JaegerDelimiter, JaegerDelimiterEncoded];
2222

23+
private const string ObsoleteMessage = "The Jaeger propagator is obsolete and will be removed in a future version. The Jaeger propagation format has been deprecated in favor of W3C Trace Context. Use TraceContextPropagator instead. See https://www.jaegertracing.io/sdk-migration/#propagation-format and https://github.com/open-telemetry/opentelemetry-specification/issues/4827 for more information.";
24+
2325
private static readonly int TraceId128BitLength = "0af7651916cd43dd8448eb211c80319c".Length;
2426
private static readonly int SpanIdLength = "00f067aa0ba902b7".Length;
2527

@@ -29,7 +31,7 @@ public class JaegerPropagator : TextMapPropagator
2931
#pragma warning restore CS0809 // Obsolete member overrides non-obsolete member
3032

3133
/// <inheritdoc/>
32-
[Obsolete("The Jaeger propagator is obsolete and will be removed in a future version. The Jaeger propagation format has been deprecated in favor of W3C Trace Context. Use TraceContextPropagator instead. See https://www.jaegertracing.io/sdk-migration/#propagation-format and https://github.com/open-telemetry/opentelemetry-specification/issues/4827 for more information.")]
34+
[Obsolete(ObsoleteMessage)]
3335
#pragma warning disable CS0809 // Obsolete member overrides non-obsolete member
3436
public override PropagationContext Extract<T>(PropagationContext context, T carrier, Func<T, string, IEnumerable<string>?> getter)
3537
#pragma warning restore CS0809 // Obsolete member overrides non-obsolete member
@@ -87,7 +89,7 @@ public override PropagationContext Extract<T>(PropagationContext context, T carr
8789
}
8890

8991
/// <inheritdoc/>
90-
[Obsolete("The Jaeger propagator is obsolete and will be removed in a future version. The Jaeger propagation format has been deprecated in favor of W3C Trace Context. Use TraceContextPropagator instead. See https://www.jaegertracing.io/sdk-migration/#propagation-format and https://github.com/open-telemetry/opentelemetry-specification/issues/4827 for more information.")]
92+
[Obsolete(ObsoleteMessage)]
9193
#pragma warning disable CS0809 // Obsolete member overrides non-obsolete member
9294
public override void Inject<T>(PropagationContext context, T carrier, Action<T, string, string> setter)
9395
#pragma warning restore CS0809 // Obsolete member overrides non-obsolete member

0 commit comments

Comments
 (0)