[Gen-art] review of draft-ietf-p2psip-drr-10.txt (resent)

Francis Dupont <Francis.Dupont@fdupont.fr> Wed, 02 October 2013 11:15 UTC

Return-Path: <Francis.Dupont@fdupont.fr>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B0FAB11E820E for <gen-art@ietfa.amsl.com>; Wed, 2 Oct 2013 04:15:18 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.67
X-Spam-Level:
X-Spam-Status: No, score=-1.67 tagged_above=-999 required=5 tests=[AWL=0.930, BAYES_00=-2.599]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id z69ue7LpABrV for <gen-art@ietfa.amsl.com>; Wed, 2 Oct 2013 04:15:12 -0700 (PDT)
Received: from givry.fdupont.fr (givry.fdupont.fr [IPv6:2001:41d0:1:6d55:211:5bff:fe98:d51e]) by ietfa.amsl.com (Postfix) with ESMTP id 9840811E8192 for <gen-art@ietf.org>; Wed, 2 Oct 2013 04:15:04 -0700 (PDT)
Received: from givry.fdupont.fr (localhost [127.0.0.1]) by givry.fdupont.fr (8.14.3/8.14.3) with ESMTP id r92BEsDt086478; Wed, 2 Oct 2013 13:14:54 +0200 (CEST) (envelope-from dupont@givry.fdupont.fr)
Message-Id: <201310021114.r92BEsDt086478@givry.fdupont.fr>
From: Francis Dupont <Francis.Dupont@fdupont.fr>
To: gen-art@ietf.org
Date: Wed, 02 Oct 2013 13:14:54 +0200
Sender: Francis.Dupont@fdupont.fr
Cc: draft-ietf-p2psip-drr.all@tools.ietf.org
Subject: [Gen-art] review of draft-ietf-p2psip-drr-10.txt (resent)
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 02 Oct 2013 11:15:19 -0000

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>.

Please resolve these comments along with any other Last Call comments
you may receive.

Document: draft-ietf-p2psip-drr-10.txt
Reviewer: Francis Dupont
Review Date: 20130927
IETF LC End Date: 20130930
IESG Telechat date: unknown

Summary: Almost Ready

Major issues: None

Minor issues: the title and the abstract must get an explicit expansion
of the RELOAD acronym, e.g., the title shoud be:
   An extension to REsource LOcation And Discovery (RELOAD) protocol
   to support Direct Response Routing

Nits/editorial comments:
 - proposed changes to the title and to the abstract (both page 1)

 - ToC page 2 and 10 page 13: Acknowledgements -> Acknowledgments

 - IMHO Authors' Addresses should be in the body so before the Appendix

 - 2 page 3 in Publicly Reachable: closed system -> closed network

 - 3.1.2 page 6 (in fact more for the RFC Editor): Figure 2 label
  should be in the same page than the figure itself (or with other
  words please avoid silly page breaks).

 - 3.2.1 page 6: the responding peer receives a response...
  if I am not fully lost it should be a request, not a response?

 - 5.1 page 8 (4 times): e.g. -> e.g.,

 - 5.2 page 9: I trust you (and the WG) for the delays...

 - 8 page 13: drat -> draft (and IMHO document is far more appropriate
  as it will be published as an RFC before this draft :-).

 - 11.2 page 14: add a reference to RFC 6887 (and see below).

 - A.1 page 16: endpoint- independent -> endpoint-independent

 - A.1 page 16: even it doesn't provide a direct way of getting
  the external assigned address (but read 11.6) PCP [RFC 6887]
  must be added to UPnP-IGD and NAT-PMP. If you need some words,
  one can consider PCP as a far more complete version of NAT-PMP.
  BTW the "test address" comment applies too to PCP.

Regards

Francis.Dupont@fdupont.fr

PS: my checker complains about standalone (-> stand-alone), inline
(?, please ignore), publically (-> publicly?, this should be addressed,
even by the RFC Editor, as it occurs many times), etc.