Skip to content

[api-baggage] revert space encoding change #5687

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

Merged
CodeBlanch merged 5 commits into from
Jun 12, 2024
Merged

[api-baggage] revert space encoding change #5687

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b14e6b3
revert space encoding change
lachmatt Jun 11, 2024
e474fa7
Update src/OpenTelemetry.Api/CHANGELOG.md
lachmatt Jun 11, 2024
6b04b9b
changelog formatting fix
lachmatt Jun 11, 2024
d8c2798
Merge branch 'main' into revert-baggage-space-encoding
CodeBlanch Jun 11, 2024
a0d4af5
Merge branch 'main' into revert-baggage-space-encoding
CodeBlanch Jun 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
changelog formatting fix
  • Loading branch information
@lachmatt
lachmatt committed Jun 11, 2024
commit 6b04b9bd94e3d1cabbe77bd78b404f8cc95d1670
3 changes: 2 additions & 1 deletion src/OpenTelemetry.Api/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@
## Unreleased

* **Breaking change:** Revert space character encoding change from `+` to `%20`
Copy link
Member

@vishweshbankwar vishweshbankwar Jun 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add this PR # as well.

Copy link
Contributor Author

@lachmatt lachmatt Jun 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added in e474fa7

Copy link
Member

@alanwest alanwest Jun 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add more information about why we're reverting this change to the changelog or PR description. Something to the effect of "... because the fix did not properly address the inconsistency with the baggage spec. We will address this issue more completely in a later release"

Copy link
Contributor Author

@lachmatt lachmatt Jun 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR description updated.

for baggage item values from [#5303](https://github.com/open-telemetry/opentelemetry-dotnet/pull/5303) [#5687](https://github.com/open-telemetry/opentelemetry-dotnet/pull/5687)
for baggage item values from [#5303](https://github.com/open-telemetry/opentelemetry-dotnet/pull/5303)
([#5687](https://github.com/open-telemetry/opentelemetry-dotnet/pull/5687))

## 1.9.0-rc.1

Expand Down