Skip to main content

Telechat Review of draft-ietf-quic-datagram-08
review-ietf-quic-datagram-08-opsdir-telechat-schoenwaelder-2022-01-31-00

Request Review of draft-ietf-quic-datagram
Requested revision No specific revision (document currently at 10)
Type Telechat Review
Team Ops Directorate (opsdir)
Deadline 2022-02-01
Requested 2022-01-18
Authors Tommy Pauly , Eric Kinnear , David Schinazi
I-D last updated 2022-01-31
Completed reviews Secdir Last Call review of -07 by Carl Wallace (diff)
Opsdir Last Call review of -07 by Jürgen Schönwälder (diff)
Genart Last Call review of -08 by Meral Shirazipour (diff)
Opsdir Telechat review of -08 by Jürgen Schönwälder (diff)
Assignment Reviewer Jürgen Schönwälder
State Completed
Request Telechat review on draft-ietf-quic-datagram by Ops Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/horafO_OUqAZfUqo1IYVjpG1_PQ
Reviewed revision 08 (document currently at 10)
Result Has nits
Completed 2022-01-31
review-ietf-quic-datagram-08-opsdir-telechat-schoenwaelder-2022-01-31-00
I have reviewed draft-ietf-quic-datagram-08 as an opsdir telechat
review. That the document is concise and generally well written.  I
have only two remarks, nothing that is blocking publication. The first
one is editorial, the second one is a question to explain the
motivation of an explicit Length field.

- In the description of the DATAGRAM frame below Figure 1, perhaps
  also describe the Type: field explicitly. Yes, the field is
  described in the text preceding Figure 1, but it is usually a good
  idea to describe all protocol fields separately, this makes it
  easier to find and quote the relevant bits and pieces.  The text
  above the figure can then likely be shortened. (Perhaps there is
  also no need to name the LEN bit by just referring to the frame
  type, or are there are frame times that have a LEN bit?)

    Type: The DATAGRAM frame type takes values 0x30 or 0x31. If the
       frame type is 0x31, the Length field is present. Otherwise, if
       the frame type is 0x30, the Length field is absent and the
       Datagram Data field extends to the end of the packet.

- Perhaps add some text motivating why having both frame type values
  is useful and detailing what implementations should do if things are
  inconsistent, e.g., the Length field is larger than Datagram Data.

While uploading	my review, I discovered that I reviewed this document
already	in December. I guess my	memory is getting faulty. Sorry for
making new comments I did not make before. ;-)