Skip to main content

Last Call Review of draft-ietf-pce-vendor-constraints-11
review-ietf-pce-vendor-constraints-11-secdir-lc-kumari-2013-12-05-00

Request Review of draft-ietf-pce-vendor-constraints
Requested revision No specific revision (document currently at 11)
Type Last Call Review
Team Security Area Directorate (secdir)
Deadline 2013-12-09
Requested 2013-11-28
Authors Fatai Zhang , Adrian Farrel
I-D last updated 2013-12-05
Completed reviews Genart Last Call review of -11 by Robert Sparks
Secdir Last Call review of -11 by Warren "Ace" Kumari
Opsdir Last Call review of -11 by Susan Hares
Assignment Reviewer Warren "Ace" Kumari
State Completed
Request Last Call review on draft-ietf-pce-vendor-constraints by Security Area Directorate Assigned
Reviewed revision 11
Result Has nits
Completed 2013-12-05
review-ietf-pce-vendor-constraints-11-secdir-lc-kumari-2013-12-05-00
Be ye not afraid...
I have reviewed this document as part of the security directorate's 
ongoing effort to review all IETF documents being processed by the 
IESG.  These comments were written primarily for the benefit of the 
security area directors.  Document editors and WG chairs should treat 
these comments just like any other last call comments.

Summary: LGTM.

Version reviewed:
Conveying Vendor-Specific Constraints in the Path Computation
Element communication Protocol
draft-ietf-pce-vendor-constraints-11.txt


Notes: I did *not* perform a formal language check. At a quick glance it looks good though.

Nits: I would like to have a table of contents. This may be a personal preference though….

While performing this review I kept thinking "Mwahaha. This can be used to carry
arbitrary information with any PCEP object that supports TLVs....  I can kvetch about the
DoS potential". But, the authors foiled my plan to rant by mentioning this in the 
Security Considerations section and even mentioning a mitigation.
Curses! Foiled again.

W




-- 
Outside of a dog, a book is your best friend, and inside of a dog, it's too dark to read