Concise Binary Object Representation (CBOR) Sequences
draft-ietf-cbor-sequence-02
Yes
(Adam Roach)
No Objection
Roman Danyliw
(Alissa Cooper)
(Alvaro Retana)
(Deborah Brungard)
(Ignas Bagdonas)
(Magnus Westerlund)
(Martin Vigoureux)
(Mirja Kühlewind)
(Suresh Krishnan)
Note: This ballot was opened for revision 01 and is now closed.
Roman Danyliw
No Objection
Warren Kumari
No Objection
Comment
(2019-09-18 for -01)
Not sent
Thank you for writing this.
Éric Vyncke
No Objection
Comment
(2019-09-12 for -01)
Sent
Carsten, Thank you for the work put into this document. I have one minor COMMENT/suggestion and I am relying on the ART area directors for the integration in the CBOR framework. Regards, -éric == COMMENTS == -- Section 2 -- C.1) Unsure whether the "(Note that, ... valid JSON documents.)" is useful in this document (I told you it is a minor comment).
Adam Roach Former IESG member
(was No Objection)
Yes
Yes
(for -01)
Not sent
Alexey Melnikov Former IESG member
Yes
Yes
(2019-09-19 for -01)
Not sent
IANA is still waiting for DE review.
Barry Leiba Former IESG member
Yes
Yes
(2019-09-11 for -01)
Sent
Thanks for a fine and mostly easy-to-read document. There’s just one bit that I find hard to read: — Section 2 — Decoding a CBOR Sequence works as follows: o If the CBOR Sequence is an empty sequence of bytes, the result is an empty sequence of CBOR data model values. o Otherwise, decode a single CBOR data item from the bytes of the CBOR sequence, and insert the resulting CBOR data model value at the start of the result of decoding the rest of the bytes as a CBOR sequence. (A streaming decoder would therefore simply deliver zero or more CBOR data model values, each of which as soon as the bytes making it up are available.) I find the phrasing of the second bullet (the part “at the start of the result of decoding the rest of the bytes as a CBOR sequence.”) really hard to parse. After a brief email exchange between Carsten and me before he zipped off on holidays, I propose this minor re-wording: NEW o Otherwise, decode a single CBOR data item from the bytes of the CBOR sequence, and insert the resulting CBOR data model value at the start of the result of repeating this decoding process recursively. (A streaming decoder would therefore simply deliver zero or more CBOR data model values, each as soon as the bytes making it up are available.) END Does that work for you, Carsten?
Benjamin Kaduk Former IESG member
Yes
Yes
(2019-09-18 for -01)
Sent
Section 2 o Otherwise, decode a single CBOR data item from the bytes of the CBOR sequence, and insert the resulting CBOR data model value at the start of the result of decoding the rest of the bytes as a CBOR sequence. (A streaming decoder would therefore simply deliver zero or more CBOR data model values, each of which as soon as the bytes making it up are available.) nit: I think s/each of which/each of which is delivered/, or just take Barry's suggestion that addresses the nit differently. Section 3 The use case for the "+cbor-seq" structured syntax suffix is the same as for "+cbor": It SHOULD be used by a media type when parsing the nit: if the use case is literally "the same as" for "+cbor" then there would seem to be literally no value in having "+cbor-seq". So perhaps "essentially the same as" or similar would be more appropriate? Section 5 I might note that when COSE is applied to the elements of a sequence, the cryptographic protection is on a per-element basis, and thus there is no guarantee of relationship between level of protection, source authentication, time of generation, etc., across members of the sequence. Section 6.1 It's probably best to treat the following as a side note and thus not an actionable comment, but couldn't the URL fragment fairly easily be used to extract numbered elements of the sequence?
Alissa Cooper Former IESG member
No Objection
No Objection
(for -01)
Not sent
Alvaro Retana Former IESG member
No Objection
No Objection
(for -01)
Not sent
Deborah Brungard Former IESG member
No Objection
No Objection
(for -01)
Not sent
Ignas Bagdonas Former IESG member
No Objection
No Objection
(for -01)
Not sent
Magnus Westerlund Former IESG member
No Objection
No Objection
(for -01)
Not sent
Martin Vigoureux Former IESG member
No Objection
No Objection
(for -01)
Not sent
Mirja Kühlewind Former IESG member
No Objection
No Objection
(for -01)
Not sent
Suresh Krishnan Former IESG member
No Objection
No Objection
(for -01)
Not sent