We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
https://github.com/w3c/baggage/blob/main/test/test_baggage.py can have more test cases added. Some can be taken from https://github.com/open-telemetry/opentelemetry-go/blob/5449f083aa4af79313d5fe5fecd554af2cb7bbbc/baggage/baggage_test.go#L239.
This can help implementers of the baggage propagator to avoid some common issues. E.g.
+
%20
CC @lachmatt @vishweshbankwar
The text was updated successfully, but these errors were encountered:
dyladan
No branches or pull requests
https://github.com/w3c/baggage/blob/main/test/test_baggage.py can have more test cases added. Some can be taken from https://github.com/open-telemetry/opentelemetry-go/blob/5449f083aa4af79313d5fe5fecd554af2cb7bbbc/baggage/baggage_test.go#L239.
This can help implementers of the baggage propagator to avoid some common issues. E.g.
+
value should be a parsed to+
(included in Go test cases)%20
key should be parsed to%20
as keys are NOT percent encoded (NOT included in Go test cases)CC @lachmatt @vishweshbankwar
The text was updated successfully, but these errors were encountered: