Skip to content

Make baggage implementation w3c spec complaint - #2167

Merged
owais merged 25 commits into
open-telemetry:mainfrom
srikanthccv:bugfix-GH2010
Oct 13, 2021
Merged

Make baggage implementation w3c spec complaint#2167
owais merged 25 commits into
open-telemetry:mainfrom
srikanthccv:bugfix-GH2010

Conversation

@srikanthccv

@srikanthccv srikanthccv commented Oct 2, 2021

Copy link
Copy Markdown
Member

Description

Updated the baggage implementation to follow the definition spec'd here https://w3c.github.io/baggage/#definition

Also Fixes #2010

@srikanthccv
srikanthccv requested a review from a team October 2, 2021 01:02
Comment thread opentelemetry-api/src/opentelemetry/baggage/propagation/__init__.py Outdated
@lzchen

lzchen commented Oct 4, 2021

Copy link
Copy Markdown
Contributor

Does this replace #2017? Can the previous PR be closed?

@srikanthccv srikanthccv added the Approve Public API check This label shows that the public symbols added or changed in a PR are strictly necessary label Oct 8, 2021
Comment thread opentelemetry-api/src/opentelemetry/baggage/__init__.py Outdated
Comment thread opentelemetry-api/src/opentelemetry/baggage/propagation/__init__.py
Co-authored-by: Leighton Chen <lechen@microsoft.com>
@owais
owais enabled auto-merge (squash) October 12, 2021 00:35
@owais
owais merged commit ed3d657 into open-telemetry:main Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approve Public API check This label shows that the public symbols added or changed in a PR are strictly necessary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Baggage value is not parsed correctly in downstream server

4 participants