Add support for b3multi #36
Comments
kuzaxak
added a commit
to x-qdo/opentelemetry-cpp-contrib
that referenced
this issue
Feb 1, 2022
Some services expect [B3 Multi][1] headers as input information. To support that we need to be able to Inject them into upstream requests. Tests will be added as a separate commit later on. Solving open-telemetry#36 [1]: https://github.com/openzipkin/b3-propagation#multiple-headers
kuzaxak
added a commit
to x-qdo/opentelemetry-cpp-contrib
that referenced
this issue
Feb 1, 2022
Some services expect [B3 Multi][1] headers as input information. To support that we need to be able to Inject them into upstream requests. Lowercase headers used to be [compatible][2] with Istio Envoy. Tests will be added as a separate commit later on. Solving open-telemetry#36 [1]: https://github.com/openzipkin/b3-propagation#multiple-headers [2]: open-telemetry/opentelemetry-go#765
kuzaxak
added a commit
to x-qdo/opentelemetry-cpp-contrib
that referenced
this issue
Sep 8, 2022
Some services expect [B3 Multi][1] headers as input information. To support that we need to be able to Inject them into upstream requests. Lowercase headers used to be [compatible][2] with Istio Envoy. Tests will be added as a separate commit later on. Solving open-telemetry#36 [1]: https://github.com/openzipkin/b3-propagation#multiple-headers [2]: open-telemetry/opentelemetry-go#765
|
@ThomsonTan Could you please have a look? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem?
Need support for b3-multi for nginx module. https://github.com/openzipkin/b3-propagation#multiple-headers
Describe the solution you'd like
Implement b3-multi in similar fashion like it is with simple b3 just do not concat trace-span-sampled into on header.
Right now header looks like
for b3-multi there should be 3 headers
The text was updated successfully, but these errors were encountered: