Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

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

Jump to bottom

Expand test coverage for baggage encoding #5500

Open
vishweshbankwar opened this issue Apr 2, 2024 · 0 comments
Open

Expand test coverage for baggage encoding #5500

vishweshbankwar opened this issue Apr 2, 2024 · 0 comments
Labels
enhancement New feature or request pkg:OpenTelemetry.Api Issues related to OpenTelemetry.Api NuGet package

Comments

@vishweshbankwar
Copy link
Member

#5260 shows a particular spec compliance issue with encoding/decoding baggage key/values.

There are some test cases available here: https://github.com/w3c/baggage/blob/main/test/test_baggage.py

We should check and see if we can incorporate them in our repo. Once the tests are implemented, we should try to fix all spec compliance issues together.
@vishweshbankwar vishweshbankwar added the enhancement New feature or request label Apr 2, 2024
@vishweshbankwar vishweshbankwar mentioned this issue Apr 2, 2024
Merged
4 tasks
@vishweshbankwar vishweshbankwar added the pkg:OpenTelemetry.Api Issues related to OpenTelemetry.Api NuGet package label Apr 3, 2024
@reyang reyang added pkg:OpenTelemetry.Api pkg:OpenTelemetry.Api Issues related to OpenTelemetry.Api NuGet package and removed pkg:OpenTelemetry.Api Issues related to OpenTelemetry.Api NuGet package pkg:OpenTelemetry.Api labels May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request pkg:OpenTelemetry.Api Issues related to OpenTelemetry.Api NuGet package
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@reyang @vishweshbankwar