opentelemetry-cpp icon indicating copy to clipboard operation
opentelemetry-cpp copied to clipboard

Published 20 hours ago • verified publisher icon open-telemetry

Decode values from `OTEL_RESOURCE_ATTRIBUTES`

Open lgfa29 opened this issue 3 years ago • 5 comments

While using the Go SDK I noticed an inconsistency in how OTEL_RESOURCE_ATTRIBUTES baggage values are encoded and decoded. This was due to a difference in the version of the W3C Baggage spec used for baggage propagation and the OTEL_RESOURCE_ATTRIBUTES decoding.

https://github.com/open-telemetry/opentelemetry-specification/pull/2670 updates the W3C Baggage spec version used by the Resource spec to require percent-decoding values from OTEL_RESOURCE_ATTRIBUTES.

lgfa29 avatar Jul 31 '22 23:07 lgfa29

This seems to be for Go SDK. Please open it here - https://github.com/open-telemetry/opentelemetry-go/issues

lalitb avatar Aug 01 '22 01:08 lalitb

My bad. This is relevant for C++ SDK too :)

lalitb avatar Aug 01 '22 01:08 lalitb

Ah yes, this started with a Go SDK change, but evolved into a spec change that affects all SDKs. Sorry for not making this clear in the original issue 😅

lgfa29 avatar Aug 02 '22 15:08 lgfa29

This issue was marked as stale due to lack of activity.

github-actions[bot] avatar Oct 15 '22 02:10 github-actions[bot]

Related: #2579

marcalff avatar Mar 07 '24 22:03 marcalff