opentelemetry-java-contrib icon indicating copy to clipboard operation
opentelemetry-java-contrib copied to clipboard

Supporting B3 baggage propagation

Open jagdishkulkarni opened this issue 4 years ago • 1 comments

Is your feature request related to a problem? Please describe. Adding support for Jaeger B3 baggage header propagation. Open Telemetry does not support baggage header propagation in a format expected by Jaeger Open tracing. Open Telemetry Java Issue-3661 and Open Telemetry Specification Issue-1971 have been raised for this. As per comments of maintainers this can be added in opentelemetry-java-contrib.

Describe the solution you'd like Add new Baggage propagator B3BaggagePropagator to make it backward compatible. This will be based on baggage header format mentioned in Jaeger B3TextMapCodec.

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

jagdishkulkarni avatar Sep 27 '21 19:09 jagdishkulkarni

Copying from https://github.com/open-telemetry/opentelemetry-specification/issues/1971#issuecomment-928179220

It was added to Jaeger (https://github.com/jaegertracing/jaeger/issues/755#issuecomment-393929718), apparently because Brave/Sleuth already used baggage-*** format, although their authors were not especially happy with it (https://github.com/openzipkin/b3-propagation/issues/22#issuecomment-391902496).

Looking at [Brave docs][1], it does not seem that baggage-*** format is actually standardized in any kind of spec, it's just the examples they used, i.e. they could've used any other naming.

yurishkuro avatar Oct 20 '21 15:10 yurishkuro