Percent character in value MUST be percent-encoded

w3c / baggage

Propagation format for distributed context: Baggage

https://w3c.github.io/baggage/

Other

46 stars 18 forks source link

Percent character in value MUST be percent-encoded #125

Closed pellared closed 5 months ago

pellared commented 6 months ago

Fixes https://github.com/w3c/baggage/issues/124

Preview | Diff

MrAlias commented 6 months ago

Doesn't the baggage-octet syntax need to be updated?

pellared commented 6 months ago

Doesn't the baggage-octet syntax need to be updated?

% is still a valid character of baggage-octet (syntax-wise). It has to be acceptable as it is needed for precent-encoding. The change is in the "semantics" how the % character is handled.

If baggage-octet would not accept % then it would be not possible to precent-encode anything.

pellared commented 6 months ago

~~I am not sure why https://labs.w3.org/repo-manager/pr/id/w3c/baggage/125 is failing.~~

EDIT:

pellared did not make IPR commitments for this group. To make the IPR commitments, pellared should join the Distributed Tracing Working Group .

Do I have to be part of https://www.w3.org/groups/wg/distributed-tracing? Maybe I should mark my PR as non-substantive?

w3cbot commented 6 months ago

pellared marked as non substantive for IPR from ash-nazg.

pellared commented 5 months ago

Bump.

@dyladan? 😉