Skip to main content

Last Call Review of draft-ietf-i2nsf-nsf-facing-interface-dm-16
review-ietf-i2nsf-nsf-facing-interface-dm-16-tsvart-lc-nishida-2021-11-21-00

Request Review of draft-ietf-i2nsf-nsf-facing-interface-dm
Requested revision No specific revision (document currently at 29)
Type Last Call Review
Team Transport Area Review Team (tsvart)
Deadline 2021-11-23
Requested 2021-11-02
Authors Jinyong Tim Kim , Jaehoon Paul Jeong , Park Jung-Soo , Susan Hares , Qiushi Lin
I-D last updated 2021-11-21
Completed reviews Opsdir Last Call review of -16 by Joe Clarke (diff)
Yangdoctors Last Call review of -08 by Acee Lindem (diff)
Genart Last Call review of -17 by Dan Romascanu (diff)
Secdir Last Call review of -16 by Kyle Rose (diff)
Tsvart Last Call review of -16 by Yoshifumi Nishida (diff)
Opsdir Telechat review of -21 by Joe Clarke (diff)
Intdir Telechat review of -21 by Jean-Michel Combes (diff)
Secdir Telechat review of -21 by Alexey Melnikov (diff)
Assignment Reviewer Yoshifumi Nishida
State Completed
Request Last Call review on draft-ietf-i2nsf-nsf-facing-interface-dm by Transport Area Review Team Assigned
Posted at https://mailarchive.ietf.org/arch/msg/tsv-art/vZZwwTuaegODR7gHlTmG57c0Fd8
Reviewed revision 16 (document currently at 29)
Result Almost ready
Completed 2021-11-21
review-ietf-i2nsf-nsf-facing-interface-dm-16-tsvart-lc-nishida-2021-11-21-00
This document has been reviewed as part of the transport area review team's
ongoing effort to review key IETF documents. These comments were written
primarily for the transport area directors, but are copied to the document's
authors and WG to allow them to address any issues raised and also to the IETF
discussion list for information.

When done at the time of IETF Last Call, the authors should consider this
review as part of the last-call comments they receive. Please always CC
tsv-art@ietf.org if you reply to or forward this review.

Summary: I think this document is almost ready for publication, but it will be
better to check the following minor points.

1: Page 48

   We don't need to support header length for TCP while supporting total length
   for UDP? I am wondering if we want to support TCP option type here.

2: Page 50:

    list total-length {
               key "start end";
               leaf start {
                 type uint32;
                 description
                   "Start udp total length for a range match.";
               }
               leaf end {
                 type uint32;
                 must '. >= ../start' {
                   error-message
                     "The end hop limit MUST be equal or greater than
                      the start hop limit.";
                 }
                 description
                   "End udp total length for a range match.";
               }

    -> is this error message correct?

3: Page 51

             leaf-list verification-tag {
               type uint32;
               description
                 "The security policy rule according to
                  udp total length.";
               reference
                 "RFC 4960: Stream Control Transmission Protocol
                            - Verification Tag";
             }

     -> Is this description correct?
     -> In my understanding, verification tag would be random values.
         I am wondering how we utilize it.

4: Page 52

   We don't need packet type for DCCP while supporting chunk types for SCTP?

5: Page 70

     <tcp>
       <destination-port-number>
        <start>5060</start>
        <start>5061</end>
       </destination-port-number>
     </tcp>

   -> should be "<end>5061</end>" ?

6: Page 72

      <tcp>
       <destination-port-number>
        <start>80</start>
        <end>80</end>
       </destination-port>
       <destination-port-number>
        <start>443</start>
        <end>443</end>
       </destination-port>
      </tcp>

  -> should be "</destination-port-number>" instead of "</destination-port>" ?

--
Yoshi