Skip to main content

Telechat Review of draft-ietf-dtn-ipn-update-11
review-ietf-dtn-ipn-update-11-artart-telechat-tiloca-2024-06-14-00

Request Review of draft-ietf-dtn-ipn-update
Requested revision No specific revision (document currently at 13)
Type Telechat Review
Team ART Area Review Team (artart)
Deadline 2024-06-18
Requested 2024-06-09
Authors Rick Taylor , Edward J. Birrane
I-D last updated 2024-06-14
Completed reviews Artart Telechat review of -11 by Marco Tiloca (diff)
Genart Last Call review of -09 by Russ Housley (diff)
Intdir Telechat review of -11 by Jean-Michel Combes (diff)
Genart Telechat review of -11 by Russ Housley (diff)
Artart Last Call review of -09 by Marco Tiloca (diff)
Opsdir Last Call review of -09 by Tim Wicinski (diff)
Assignment Reviewer Marco Tiloca
State Completed
Request Telechat review on draft-ietf-dtn-ipn-update by ART Area Review Team Assigned
Posted at https://mailarchive.ietf.org/arch/msg/art/AqPtczYb3e3Lor2N3hRvNsvoNuY
Reviewed revision 11 (document currently at 13)
Result Ready w/nits
Completed 2024-06-14
review-ietf-dtn-ipn-update-11-artart-telechat-tiloca-2024-06-14-00
I reviewed this document as part of the Applications and Real-Time (ART) Area
Review Team's ongoing effort to review all IETF documents being processed by
the IESG. These comments were written primarily for the benefit of the ART Area
Directors. Document authors, document editors, and WG Chairs should treat these
comments just like any other IETF Last Call comments.

I previously reviewed version -09, then the following version -10 addressed my
comments. This latest version -11 looks good, I have only one minor comment
below.

Best,
/Marco

[Section 5.2]

* In Section 4, bullet point 2, there has been an understandable change from
"MUST" to "MAY", as to the omission of the '0' Allocator Identifier from the
textual encoding of an ipn URI.

  Consistent with that change, I think that the last sentence in Section 5.2
  should be updated as follows.

  OLD:
  This means that the URIs dtn:none (Section 4.2.5.1.1 of [RFC9171]) and
  ipn:0.0 both refer to the BPv7 'null' endpoint.

  NEW:
  This means that the URIs dtn:none (Section 4.2.5.1.1 of [RFC9171]), ipn:0.0,
  and ipn:0.0.0 all refer to the BPv7 'null' endpoint.

[Nits]

* Section 7.2
- s/Mitigation for/The used mitigation for