w3c / baggage

Propagation format for distributed context: Baggage
https://w3c.github.io/baggage/
Other
48 stars 18 forks source link

Precent character in value must be precent-encoded #124

Closed pellared closed 8 months ago

pellared commented 9 months ago

https://github.com/w3c/baggage/blob/ebda53c1d33031d95f1104f44840d3d4b875eb89/baggage/HTTP_HEADER_FORMAT.md?plain=1#L53-L56

This misses a case that precent character MUST be encoded as well.

Otherwise how we would know if %20 stands for ` (whitespace) or%20?%20value must be encoded into%2520`.

Found here: https://github.com/open-telemetry/opentelemetry-go/pull/4756#discussion_r1432889535