Last Call Review of draft-ietf-isis-remaining-lifetime-02
review-ietf-isis-remaining-lifetime-02-genart-lc-holmberg-2016-08-06-00
| Request | Review of | draft-ietf-isis-remaining-lifetime |
|---|---|---|
| Requested revision | No specific revision (document currently at 04) | |
| Type | Last Call Review | |
| Team | General Area Review Team (Gen-ART) (genart) | |
| Deadline | 2016-08-15 | |
| Requested | 2016-08-01 | |
| Authors | Les Ginsberg , Paul Wells , Bruno Decraene , Tony Przygienda , Hannes Gredler | |
| Draft last updated | 2016-08-06 | |
| Completed reviews |
Genart Last Call review of -02
by
Christer Holmberg
(diff)
Secdir Last Call review of -01 by Rich Salz (diff) Opsdir Last Call review of -01 by Tim Wicinski (diff) |
|
| Assignment | Reviewer | Christer Holmberg |
| State | Completed | |
| Review |
review-ietf-isis-remaining-lifetime-02-genart-lc-holmberg-2016-08-06
|
|
| Reviewed revision | 02 (document currently at 04) | |
| Result | Almost Ready | |
| Completed | 2016-08-06 |
review-ietf-isis-remaining-lifetime-02-genart-lc-holmberg-2016-08-06-00
I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq> Document: draft-ietf-isis-remaining-lifetime-02 Reviewer: Christer Holmberg Review Date: 6 August 2016 IETF LC End Date: 15 August 2016 IETF Telechat Date: N/A Summary: The document is well written, and almost ready for publication as a standards track RFC, but I have a couple of editorial comments that I’d like the authors to address. Major Issues: None Minor Issues: None Editorial Issues: The Abstract says: “Corruption of the Remainining Lifetime Field in a Link State PDU can go undetected. In certain scenarios this may cause or exacerbate flooding storms. It is also a possible denial of service attack vector. This document defines a backwards compatible solution to this problem.” …and the first sentence of the Problem Statement says: “Each Link State PDU (LSP) includes a Remaining Lifetime field.” I have no idea what a Link State PDU is, and there is no introduction to what the draft is all about. The text jumps direction into the work. So, please add a reference to Link State PDU (LSP), and please give a little bit of introduction text what context/environment this is all about. I assume there is some core document which describes the context/environment where the Link State PDU is used?