Skip to main content

Last Call Review of draft-ietf-dots-signal-filter-control-04
review-ietf-dots-signal-filter-control-04-genart-lc-holmberg-2020-06-06-00

Request Review of draft-ietf-dots-signal-filter-control
Requested revision No specific revision (document currently at 07)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2020-06-15
Requested 2020-06-01
Authors Kaname Nishizuka , Mohamed Boucadair , Tirumaleswar Reddy.K , Takahiko Nagata
Draft last updated 2020-06-06
Completed reviews Opsdir Last Call review of -04 by Tim Chown (diff)
Secdir Last Call review of -04 by Shawn M Emery (diff)
Genart Last Call review of -04 by Christer Holmberg (diff)
Assignment Reviewer Christer Holmberg
State Completed
Review review-ietf-dots-signal-filter-control-04-genart-lc-holmberg-2020-06-06
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/tQEIQ9-oGcskGARHKG5pVOF6TZA
Reviewed revision 04 (document currently at 07)
Result Ready with Nits
Completed 2020-06-06
review-ietf-dots-signal-filter-control-04-genart-lc-holmberg-2020-06-06-00
I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-dots-signal-filter-control-04
Reviewer: Christer Holmberg
Review Date: 2020-06-06
IETF LC End Date: 2020-06-15
IESG Telechat date: Not scheduled for a telechat

Summary: The document is well written, and pretty much ready for publication. I
do have a couple of minor editorial comments that I'd like the authors to
address.

Major issues: None

Minor issues: None

Nits/editorial comments:

Q1:  Please expand DOTS on first occurence.

---

Q2:  The Security Considerations say:

   "This specification does not allow to create new filtering rules,
   which is the responsibility of the DOTS data channel."

Unless I missed it, I think it would be useful to state this also earlier in
the document, e.g., in the Introduction.

---

Q3: The Security Consideration say:

   "The security considerations discussed in
   [I-D.ietf-dots-signal-channel] and [I-D.ietf-dots-data-channel] need
   to be taken into account."

I think it is obvious that those security considerations need to be taken into
account. I would suggest to re-phrase, and say something like:

   "The generic security considerations for DOTS signal channels are
   defined in   [I-D.ietf-dots-signal-channel]. The generic security
   considerations for DOTS data channels are defined in
   [I-D.ietf-dots-data-channel]. This Section defines the security
   considerations that are specific to the DOTS extension defined
   in this document."