Sensor Measurement Lists (SenML) Fields for Indicating Data Value Content-Format
draft-ietf-core-senml-data-ct-07
Yes
Francesca Palombini
No Objection
Erik Kline
John Scudder
Warren Kumari
(Alvaro Retana)
(Martin Duke)
(Martin Vigoureux)
Note: This ballot was opened for revision 05 and is now closed.
Francesca Palombini
Yes
Erik Kline
No Objection
John Scudder
No Objection
Murray Kucherawy
(was Discuss)
No Objection
Comment
(2021-09-22 for -06)
Sent
Thanks to Bron Gondwana for the ARTART review.
Roman Danyliw
No Objection
Comment
(2021-09-18 for -05)
Sent
Is there any generic guidance to provide on expected error handling behavior if: -- ct (or bct) contains a number outside of the 0-65535 range, or an unregistered value per the CoAP Content-Formats? -- ct (or bct) contains a text value that is not a registered Content-Format-String? -- ct (or bct) contains a valid Content-Format-String, but an unregistered Content-Format? Or is this application specific?
Warren Kumari
No Objection
Alvaro Retana Former IESG member
No Objection
No Objection
(for -05)
Not sent
Benjamin Kaduk Former IESG member
(was Discuss)
No Objection
No Objection
(2021-10-13 for -06)
Sent
Thanks for the updates in the -06 and the discussion to get there. I'm happy with where things ended up, and have just one comment after reading the diff: The ABNF comment for the "Cleaned up" entries currently says "leaving the parameter as mandatory", which is only true in a very specific sense; "leaving the parameter as mandatory within the grouping" or "leaving the parameter as mandatory when the separator is present" might be more clear to a reader not focusing on the specific context of the statement.
Lars Eggert Former IESG member
No Objection
No Objection
(2021-09-20 for -05)
Sent
The references to IANA registries could be informative, since RFC7252 is already normatively cited. ------------------------------------------------------------------------------- All comments below are about very minor potential issues that you may choose to address in some way - or ignore - as you see fit. Some were flagged by automated tools (via https://github.com/larseggert/ietf-reviewtool), so there will likely be some false positives. There is no need to let me know what you did with these suggestions. Section 4. , paragraph 2, nit: > t" fields (explanation/comments in parenthesis): * "60" (CoAP Content-Format > ^^^^^^^^^^^^^^ Did you mean "in parentheses"? "parenthesis" is the singular. These URLs in the document can probably be converted to HTTPS: * http://www.iana.org/assignments/senml * http://www.iana.org/assignments/core-parameters * http://www.iana.org/assignments/http-parameters * http://www.iana.org/assignments/media-types
Martin Duke Former IESG member
No Objection
No Objection
(for -05)
Not sent
Martin Vigoureux Former IESG member
No Objection
No Objection
(for -05)
Not sent
Robert Wilton Former IESG member
No Objection
No Objection
(2021-09-21 for -05)
Sent
Thanks for this document. I have to confess that I am less convinced of the need to have two ways to represent this information, i.e., to also support Content-Format-Strings as opposed to always requiring a entry in the CoAP Content-Formats registry. The CoAP Content-Formats registry seems to be of reasonable size and have a lot of space available for FCFS allocations which would mean that it should be quite easy to extend if required. However, I'm willing to defer to the authors & CORE WG expertise here supporting both ways are needed/useful. Regards, Rob