ioggstream / draft-polli-resource-digests-http

THIS REPO WAS MOVED TO https://github.com/httpwg/http-extensions/
https://httpwg.org/http-extensions/draft-ietf-httpbis-digest-headers.html
Other
0 stars 1 forks source link

Clarify digest-algorithm syntax #34

Closed ioggstream closed 5 years ago

ioggstream commented 5 years ago

I expect

https://github.com/ioggstream/draft-polli-resource-digests-http/blame/master/draft-polli-resource-digests-http.md#L302

ioggstream commented 5 years ago

Taken from https://github.com/martinthomson/http-mice/blame/537cc8b7f61787306d358f14178b146eae631ee4/draft-thomson-http-mice.md#L245

The MI header field uses the extended ABNF syntax defined in Section 1.2 of
[RFC7230] and the `parameter` rule from [RFC7231]:

we could say that the Digest header takes parameter, instead of the digest-algorithm: this would be similar to Want-Digest

Digest: new-algo=f1b2c3;rs=5, sha-256=fbfbfba
LPardue commented 5 years ago

If I understand correctly, this seems like two indpendent issues:

1) Ensure that the syntax definition aligns with modern best practice. Whether that be RFC723x, HTTP core, or Structured headers.

2) Consider adding parameters to the Digest header. Do we have any examples in the wild that do this, or use cases that want to do this?

ioggstream commented 5 years ago

I try to clarify. RFC3230 stated that:

   For some algorithms, one or more parameters may be
   supplied.

      digest-algorithm = token

   The BNF for "parameter" is as is used in RFC 2616 [4].  All digest-
   algorithm values are case-insensitive.

but:

In this draft we preserved retained that sentence but:

So, about your questions:

ioggstream commented 5 years ago

moved to https://github.com/httpwg/http-extensions/issues/850