Re: [Gen-art] Gen-art review of draft-gont-intarea-obsolete-eid-option-01

Fernando Gont <fgont@si6networks.com> Mon, 12 November 2012 18:43 UTC

Return-Path: <fgont@si6networks.com>
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 946E521F8790; Mon, 12 Nov 2012 10:43:51 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.599
X-Spam-Level:
X-Spam-Status: No, score=-2.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id UpIDCMdGwpQR; Mon, 12 Nov 2012 10:43:50 -0800 (PST)
Received: from web01.jbserver.net (web01.jbserver.net [93.186.182.34]) by ietfa.amsl.com (Postfix) with ESMTP id 967E421F878E; Mon, 12 Nov 2012 10:43:50 -0800 (PST)
Received: from [186.134.15.187] (helo=[192.168.123.122]) by web01.jbserver.net with esmtpsa (TLSv1:DHE-RSA-CAMELLIA256-SHA:256) (Exim 4.80.1) (envelope-from <fgont@si6networks.com>) id 1TXyz3-0001sp-C6; Mon, 12 Nov 2012 19:43:45 +0100
Message-ID: <50A14350.9070705@si6networks.com>
Date: Mon, 12 Nov 2012 15:43:28 -0300
From: Fernando Gont <fgont@si6networks.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121028 Thunderbird/16.0.2
MIME-Version: 1.0
To: "Moriarty, Kathleen" <kathleen.moriarty@emc.com>
References: <182B306DD37FAD4882AB1C1EBE2CA9020DCDDF9129@MX15A.corp.emc.com>
In-Reply-To: <182B306DD37FAD4882AB1C1EBE2CA9020DCDDF9129@MX15A.corp.emc.com>
X-Enigmail-Version: 1.4.5
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7bit
X-Mailman-Approved-At: Mon, 12 Nov 2012 11:31:19 -0800
Cc: "draft-gont-intarea-obsolete-eid-option.all@tools.ietf.org" <draft-gont-intarea-obsolete-eid-option.all@tools.ietf.org>, "gen-art@ietf.org" <gen-art@ietf.org>, "iesg@ietf.org" <iesg@ietf.org>
Subject: Re: [Gen-art] Gen-art review of draft-gont-intarea-obsolete-eid-option-01
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: Mon, 12 Nov 2012 18:43:51 -0000

HI, Kathleen,

Thnas so much for your feedback. Please find my responses in-line...

On 10/25/2012 05:36 PM, Moriarty, Kathleen wrote:
> Summary:  The document needs a few small revisions. This document
> formally deprecates the IPv6 Endpoint Identification (EID) option
> (hex value 0x8a).

FWIW, I'll update the title to note that this in an IPv6 option.



> Major issues:
> 
> Minor issues: RFC2119 is referenced, but not used.  I don't see the
> use of any RFC2119 language in the document, so it may be safe to
> remove this reference.

Will do.



> I don't think the reference for EID option is valid.  I also looked
> at the reference in the IANA page and that no longer contains a link.
> From a google search, it looks like this was an internet draft that
> was never published as an RFC and expired,
> http://ana-3.lcs.mit.edu/~jnc/nimrod/eidoption.txt Listing it as "A
> work in progress" when it is an expired draft s not appropriate.

Ok, I will update this references as indicated.



> Nits/editorial comments:
> 
> I recommend changing the following portion of the abstract to state
> what is accomplished in this draft as the language is a little
> confusing: "possibly serving as a basis for providing advice" Since
> this draft doesn't actually provide the advice, just says it may
> provide a basis for providing the advice, I would recommend leaving
> this out of the abstract.

I've already applied such change -- thanks!



> Recommend removing "e.g." fromt he last sentence int he Security
> Considerations section as it is not needed. Change from: "However,
> formally deprecating this option may serve as a basis for e.g.
> providing advice about filtering packets containing such option (in a
> similar way to [I-D.ietf-opsec-ip-options-filtering] for the IPv4
> case)." TO: "However, formally deprecating this option may serve as a
> basis for providing advice about filtering packets containing such
> option (in a similar way to [I-D.ietf-opsec-ip-options-filtering] for
> the IPv4 case)."

I've removed such sentence altogether, upon suggestion by Wes Eddy.

Thanks!

Best regards,
-- 
Fernando Gont
SI6 Networks
e-mail: fgont@si6networks.com
PGP Fingerprint: 6666 31C6 D484 63B2 8FB1 E3C4 AE25 0D55 1D4E 7492