w3c / baggage

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

Use RFC 7230 for token definition #79

Closed dyladan closed 2 years ago

dyladan commented 2 years ago

Resolves the issue raised by https://github.com/w3ctag/design-reviews/issues/650#issuecomment-969290866

Changes the definition of token to refer to RFC 7230 instead of RFC 2616. RFC 7230 is already used for the definition of OWS and for header concatenation rules. IMO the new definition is more clear and easily readable.

Note: The definitions are written differently. The old 2616 definition defines token in terms of what characters ARE NOT allowed (separators). The new 7230 definition defines token in terms of which characters ARE allowed. The definitions appear to be identical as far as I can tell. There are 72 allowed characters in the US ASCII range (0-127).

Old token definition from 2616:

       CHAR           = <any US-ASCII character (octets 0 - 127)>
       CTL            = <any US-ASCII control character
                        (octets 0 - 31) and DEL (127)>

       token          = 1*<any CHAR except CTLs or separators>
       separators     = "(" | ")" | "<" | ">" | "@"
                      | "," | ";" | ":" | "\" | <">
                      | "/" | "[" | "]" | "?" | "="
                      | "{" | "}" | SP | HT

New token definition from 7230:

     VCHAR defined as any visible [USASCII] character

     token          = 1*tchar

     tchar          = "!" / "#" / "$" / "%" / "&" / "'" / "*"
                    / "+" / "-" / "." / "^" / "_" / "`" / "|" / "~"
                    / DIGIT / ALPHA
                    ; any VCHAR, except delimiters

Preview | Diff

dyladan commented 2 years ago

The repository manager does not recognize me as a contributor, and when I click "revalidate" it gives me the following error:

image

Could there be some sort of outage?