mirror of
https://github.com/open-telemetry/opentelemetry-go.git
synced 2025-02-01 13:07:51 +02:00
bdb9322ebd
* Use url.PathUnescape rather than url.QueryUnescape I believe this addresses the majority of the cases described in https://github.com/open-telemetry/opentelemetry-go/issues/3601 Golang's url.QueryUnescape will render url _path_ elements (e.g. /, +) as spaces: `foo+bar` is rendered as `foo bar`. Path elements are (as I read the spec) legal W3C baggage values, and replacing them with spaces fails the value validation regex. url.PathEscape allows path elements through unmolested. Signed-off-by: Nathan J. Mehl <n@oden.io> * Update CHANGELOG.md address comments Co-authored-by: Robert Pająk <pellared@hotmail.com> --------- Signed-off-by: Nathan J. Mehl <n@oden.io> Co-authored-by: Robert Pająk <pellared@hotmail.com>