Re: Fwd: Re: Gen-ART review of draft-ietf-ccamp-assoc-ext-04

Lou Berger <lberger@labn.net> Thu, 30 August 2012 14:58 UTC

Return-Path: <lberger@labn.net>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 5A9C721F851A for <ietf@ietfa.amsl.com>; Thu, 30 Aug 2012 07:58:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -99.426
X-Spam-Level:
X-Spam-Status: No, score=-99.426 tagged_above=-999 required=5 tests=[AWL=0.735, BAYES_00=-2.599, FH_RELAY_NODNS=1.451, HELO_MISMATCH_COM=0.553, IP_NOT_FRIENDLY=0.334, RDNS_NONE=0.1, USER_IN_WHITELIST=-100]
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 PWbGjSiNjpXS for <ietf@ietfa.amsl.com>; Thu, 30 Aug 2012 07:58:06 -0700 (PDT)
Received: from oproxy6-pub.bluehost.com (oproxy6.bluehost.com [IPv6:2605:dc00:100:2::a6]) by ietfa.amsl.com (Postfix) with SMTP id 607B121F84C9 for <ietf@ietf.org>; Thu, 30 Aug 2012 07:58:06 -0700 (PDT)
Received: (qmail 17788 invoked by uid 0); 30 Aug 2012 14:58:05 -0000
Received: from unknown (HELO box313.bluehost.com) (69.89.31.113) by cpoproxy3.bluehost.com with SMTP; 30 Aug 2012 14:58:05 -0000
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=labn.net; s=default; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:References:Subject:CC:To:MIME-Version:From:Date:Message-ID; bh=lB8FsGTzYwD2luv33h+mULijVBFzJURM6SjeDTWnkVA=; b=fJA6vMJ5u8/BIIYSFsANBozw4RWMh2hvG7BE7Pky0TjwFoysKxtYolIbQP8HgERKu++ZAuFEwME1UhSWA2DDe++dQjoQQoTyIzP9D5tfPYI3fw3ahjQARuvUW6hL7/W5;
Received: from box313.bluehost.com ([69.89.31.113]:32973 helo=[127.0.0.1]) by box313.bluehost.com with esmtpa (Exim 4.76) (envelope-from <lberger@labn.net>) id 1T76C5-0005fU-GM; Thu, 30 Aug 2012 08:58:05 -0600
Message-ID: <503F7F7D.8080404@labn.net>
Date: Thu, 30 Aug 2012 10:58:05 -0400
From: Lou Berger <lberger@labn.net>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/20120824 Thunderbird/15.0
MIME-Version: 1.0
To: Adrian Farrel <adrian@olddog.co.uk>, ietf@ietf.org
Subject: Re: Fwd: Re: Gen-ART review of draft-ietf-ccamp-assoc-ext-04
References: <503F77CF.1080005@labn.net> <503F7857.3070905@labn.net>
In-Reply-To: <503F7857.3070905@labn.net>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
X-Identified-User: {1038:box313.bluehost.com:labnmobi:labn.net} {sentby:smtp auth 69.89.31.113 authed with lberger@labn.net}
Cc: draft-ietf-ccamp-assoc-ext.all@tools.ietf.org, gen-art@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 30 Aug 2012 14:58:07 -0000

Eek,
     I somehow managed to broadcast this!  My apologies.

Lou

On 8/30/2012 10:27 AM, Lou Berger wrote:
>
> Adrian,
>       Shout (or change the ID state) when you're ready for the update to
> be submitted.
>
> Thanks,
> Lou
> -------- Original Message --------
> Subject: 	Re: Gen-ART review of draft-ietf-ccamp-assoc-ext-04
> Date: 	Thu, 30 Aug 2012 10:25:19 -0400
> From: 	Lou Berger <lberger@labn.net>
> To: 	Peter Yee <peter@akayla.com>
> CC: 	draft-ietf-ccamp-assoc-ext.all@tools.ietf.org, gen-art@ietf.org,
> ietf@ietf.org
>
>
>
> Peter,
> 	Thank you for the comments.  Please see below for responses in-line.
>
>
> On 8/29/2012 11:31 PM, Peter Yee wrote:
>> 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-ccamp-assoc-ext-04
>> Reviewer: Peter Yee
>> Review Date: Aug-28-2012
>> IETF LC End Date: Aug-29-2012
>> IESG Telechat date: Not known
>>
>> Summary: This draft is basically ready for publication, but has nits that
>> should be
>> fixed before publication. [Ready with nits.]
>>
>> This document provides extensions to the scope of use of the RSVP
>> ASSOCIATION
>> object as well as providing an extended ASSOCIATION object capable of
>> handling
>> a longer Association ID.
>>
>> Nits:
>>
>> In the last example (Symmetric NAT), last sentence: "mechanisms" ->
>> "mechanism".
>>
>> Section 2, 4th paragraph (the replacement text): "the the" -> "the".
>>
>> Section 3.2.1, 2nd paragraph, 1st sentence: "are" -> "is".  Alternatively,
>> you
>> could change "format" to "formats".
>>
>> Section 3.2.2, 1st sentence: "apply" -> "applies".
>>
>> Section 4.2, 1st sentence: "a" -> "an" in both occurrences.
>>
>> Section 4.2, last paragraph, 2nd sentence: "a" -> "an".
>>
> Thank you!
>
>> Questions:
>>
>> These are questions you may wish to answer but the draft is acceptable
>> without response:
>>
>> 1) In Section 4.2, 4th bullet, is there any implied relationship between the
>> Extended Association ID and the Association ID?  Or are they independent
>> values that simply must be matched?
> The latter (as the text says.)
>
>> 2) Section 4.2, 5th bullet, you make a first and only mention of padding
>> bytes.
>> Are you using a specific method for generating these padding bytes or are
>> they random?
> No, as it is completely up to the creator of the object to use it
> consistently.
>
>> Given the matching requirement on ASSOCIATION objects,
>> it might be best to specify the padding generation so that if the object is
>> regenerated, it will still be matched by intermediary nodes.  I've presumed
>> that the padding bytes are for meeting the 4-byte multiple requirement,
>> but I don't know if implementations would ever be free to regenerate the
>> object for subsequent transmissions of that object.
> I'll add ", without modification," to the last sentence of 3.1.2 and 3.2.2.
>
> Thank you for the comments!
>
> Lou
>
>
>
>