Re: Submitted new I-D: Cache Digests for HTTP/2

On 2016-01-18 11:19, Stefan Eissing wrote:
> +1 generic header parameter form.
>
> If I may make a proposal:
>
> Cache-Digest      = "Cache-Digest" ":" #digest-value
>    digest-value    = "<" base64url encoded digest ">" *( ";" digest-param )
>    digest-param    = ( ( "domain" "=" domain-value )
>                    | ( "path" "=" path-value )
>                    | ( "codec" "=" codec-value )
>                    | ( "update" ( "=" update-param ) )
>                    | ( digest-extension ) )
>
>   digest-extension = ( parmname [ "=" ( ptoken | quoted-string ) ] )
>
>   domain-value     =
>         authority              # defined in RFC3986
>         | wildcard-identifier  # defined in RFC 6125 6.4.3
>
>   path-value       = path-absolute    # defined in RFC3986 3.3
>
>   codec            = ( "GCS-SHA256" | ( ptoken | quoted-string ) )
> ...

I wouldn't use angle brackets here; let's leave them for use in places 
where the value is a URI (Link header field and several WebDAV header 
fields).

Just us token/quoted-sting here.

Also, do not wire parameter names into the ABNF; this mixes syntax with 
semantics.

Best regards, Julian

Received on Monday, 18 January 2016 10:33:06 UTC