Skip to main content

Last Call Review of draft-ietf-p2psip-share-09
review-ietf-p2psip-share-09-genart-lc-miller-2016-10-25-00

Request Review of draft-ietf-p2psip-share
Requested revision No specific revision (document currently at 10)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2016-10-25
Requested 2016-10-13
Authors Alexander Knauf , Thomas C. Schmidt , Gabriel Hege , Matthias Wählisch
I-D last updated 2016-10-25
Completed reviews Genart Last Call review of -09 by Matthew A. Miller (diff)
Secdir Early review of -08 by Russ Housley (diff)
Secdir Last Call review of -09 by Russ Housley (diff)
Opsdir Last Call review of -09 by Rick Casarez (diff)
Assignment Reviewer Matthew A. Miller
State Completed
Request Last Call review on draft-ietf-p2psip-share by General Area Review Team (Gen-ART) Assigned
Reviewed revision 09 (document currently at 10)
Result Ready w/nits
Completed 2016-10-25
review-ietf-p2psip-share-09-genart-lc-miller-2016-10-25-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

< 

http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq

 >.

Document: draft-ietf-p2psip-share-09
Reviewer: Matthew A. Miller
Review Date: 2016-11-02
IETF LC End Date: 2016-10-25
IESG Telechat date: 2016-11-03

Summary:

This document is ready to be published as a Standards Track
document once the nits are addressed.

Major issues: NONE

Minor issues: NONE

Nits/editorial comments:

* idnits reports a stale reference to I-D.ietf-p2psip-sip (should
  be RFC 7904).

* In 5.1. "Overview", the word "witch" should be "which".

* In 5.3. "Overlay Configuration Document Extension", there should
  be a space between "P2PSIP" and "[I-D.ietf-p2psip-sip]".

* In 6.2. "Revoking White Access", there should be a space between
  "see" and "[RFC6940]".

* In 6.4. "Operations of Storing Peers", a comma is missing between
  "peers" and "at" in the phrase "Storing peers at which Shared
  Resource and ACL are physically stored ...".


Non-issue comments:

* idnits is reporting weird spacing and "possible code", but that
  appears to be due to the Relax NG grammar.  In my opinion the nit
  can be safely ignored.


-- 
- m&m

Matt Miller
Cisco Systems, Inc.



Attachment:


signature.asc




Description:

 OpenPGP digital signature