Skip to content

feat: add zipkin b3 header propagation#1110

Merged
agrawroh merged 1 commit intoenvoyproxy:mainfrom
isker:b3-propagation
Apr 19, 2026
Merged

feat: add zipkin b3 header propagation#1110
agrawroh merged 1 commit intoenvoyproxy:mainfrom
isker:b3-propagation

Conversation

@isker
Copy link
Copy Markdown
Contributor

@isker isker commented Apr 15, 2026

We unfortunately chose the wrong path down the tech tree many years ago and now we are stuck on zipkin header propagation throughout our service call stack. Configure the tracer here to parse and propagate zipkin's b3 headers in addition to the more standard otel/w3c ones.

This isn't gated behind configuration as I think the risk is small; this could only break someone's usage if they were already propagating both sets of headers to this service, and those headers conflicted. Given that Envoy is the typical client of this service, and I don't think you can even configure Envoy to do multiple types of propagation simultaneously, I think the risk is low. (There is no risk the other way as this service doesn't propagate headers to any place.)

We unfortunately chose the wrong path down the tech tree many years ago
and now we are stuck on zipkin header propagation throughout our service
call stack. Configure the tracer here to parse and propagate zipkin's b3
headers in addition to the more standard otel/w3c ones.

This isn't gated behind configuration as I think the risk is small; this
could only break someone's usage if they were already propagating both
sets of headers to this service, and those headers conflicted. Given
that Envoy is the typical client of this service, and I don't think you
can even configure Envoy to do multiple types of propagation
simultaneously, I think the risk is low. (There is no risk the other way
as this service doesn't propagate headers to any place.)

Signed-off-by: Ian Kerins <[email protected]>
@agrawroh agrawroh merged commit 4ffd8a2 into envoyproxy:main Apr 19, 2026
6 checks passed
@isker
Copy link
Copy Markdown
Contributor Author

isker commented Apr 19, 2026

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants