Re: [radext] [Gen-art] Genart last call review of draft-ietf-radext-coa-proxy-05

Alissa Cooper <alissa@cooperw.in> Wed, 15 August 2018 18:34 UTC

Return-Path: <alissa@cooperw.in>
X-Original-To: radext@ietfa.amsl.com
Delivered-To: radext@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 9723613107B; Wed, 15 Aug 2018 11:34:54 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.7
X-Spam-Level:
X-Spam-Status: No, score=-2.7 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=unavailable autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=cooperw.in header.b=np+ajRdg; dkim=pass (2048-bit key) header.d=messagingengine.com header.b=Y8ocFlSR
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id kbMrz3zKkGd7; Wed, 15 Aug 2018 11:34:51 -0700 (PDT)
Received: from wout2-smtp.messagingengine.com (wout2-smtp.messagingengine.com [64.147.123.25]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D59FA130E0A; Wed, 15 Aug 2018 11:34:51 -0700 (PDT)
Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.west.internal (Postfix) with ESMTP id C8E702DA; Wed, 15 Aug 2018 14:34:50 -0400 (EDT)
Received: from mailfrontend1 ([10.202.2.162]) by compute7.internal (MEProxy); Wed, 15 Aug 2018 14:34:51 -0400
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cooperw.in; h=cc :content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc; s=fm3; bh=K5Ff+d+2Dhxq9Nxx3OBOLKdS8mkqu nkHGs8XSDvuI1U=; b=np+ajRdg6xSGh0yRwil3naVvXre5xSzraJk30PZxUuc2x Q4Z3wX5jpQZfTiPUxTOHzrZkSKVhM69D27ydJftZI+sTZKk2pWjk+/WxwkCkD4Am bR43FB8PWX427ejfEwhtmjoFu2efBA5Gi/+FNSEBvaKXE2pdqqCznj94sGMvUwwS diFFjV+t8hBAYF17cELXcI74SeuXSVj6vMF3dE01t4+kmlWVZBlxnTMi1D4Ucf/4 YzGZHN5rS8GFQASd3WVeabT+rGlRUVL8gLejAMyCRYx7Az8D7CP6Fzgi4jaiKBxz LfpM2EoEWCrIrPSJyiT7jU2N0TO+p9BLtHI52Pejg==
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; bh=K5Ff+d +2Dhxq9Nxx3OBOLKdS8mkqunkHGs8XSDvuI1U=; b=Y8ocFlSRUjeVgEkoVgbCO6 R/FxEiQ/8x+qW0xNz9ffMofb+97a/6lKxm46q/XO88p8BfaPT2x/KL21T72NRu2I 3JlPd88c8zeeXbWJN1KT3Dan8JlFkJfjVDh2CB5l2Hj+T5cQzZw1n6bYvPeIUCa2 /GfteTse7k/5BDYqpwT1qslY1lugI6pGlg1lxOp5gIVW/2c8DyJOJSue0xhWOYLy vzjCmEcPOgwtoVt/W8/yXcAeVKwCxv5pwzR2uREQBuFcJXKFVO1ZIQS1TTOclU9W xJGwrQUlSiec19+bvRC303GA5RuGFeTJ4f+8XQhuRCKQqCHAYEAnKnnXj+2TvSKQ ==
X-ME-Proxy: <xmx:SXJ0W54oiI1A_JjjTnd7-m3xlTCxzk6cU7sfoA0DvBW5LeCDw_opzA> <xmx:SXJ0WwV1YLrT-t744o3r_81UOhKoRVuaCfselc1T8eVuRO0xcy_dqw> <xmx:SXJ0WwCfZokItXLweGBT-J8ykeZb0P16kJgJozd17McX04NYnVEbng> <xmx:SXJ0W37Gds86FSISbYagG2PGjl8moONhGxhtSWeMok0Hhb_qsKvPPg> <xmx:SXJ0WzyePGcEK3O1UllY8ZyXFAIEG5S5KkKub1k-RpCn36gt_Zcumg> <xmx:SnJ0W-Ok1QkLY5JXHXCD_tSVGLZQVAQGAzMVDbXUY1FJoNY-0v30bQ>
X-ME-Sender: <xms:SXJ0W5bExcu1f-No4J1I8Y_iG052IM_0wjFLdlzogwJQ-d3Rtw3yCA>
Received: from dhcp-10-150-9-145.cisco.com (unknown [173.38.117.87]) by mail.messagingengine.com (Postfix) with ESMTPA id 74F45E463E; Wed, 15 Aug 2018 14:34:49 -0400 (EDT)
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 11.5 \(3445.9.1\))
From: Alissa Cooper <alissa@cooperw.in>
In-Reply-To: <3333032E-47BC-4AF9-97D5-F02F7FC260D8@deployingradius.com>
Date: Wed, 15 Aug 2018 14:34:48 -0400
Cc: Tim Evens <tievens=40cisco.com@dmarc.ietf.org>, radext@ietf.org, gen-art <gen-art@ietf.org>, draft-ietf-radext-coa-proxy.all@ietf.org
Content-Transfer-Encoding: quoted-printable
Message-Id: <6B111A9A-A84D-4CE2-A6E9-E4059784195C@cooperw.in>
References: <153420629857.24982.2722057727598250846@ietfa.amsl.com> <3333032E-47BC-4AF9-97D5-F02F7FC260D8@deployingradius.com>
To: Alan DeKok <aland@deployingradius.com>
X-Mailer: Apple Mail (2.3445.9.1)
Archived-At: <https://mailarchive.ietf.org/arch/msg/radext/SCSOxXJpyc6wM-c2S4R37jpAifU>
Subject: Re: [radext] [Gen-art] Genart last call review of draft-ietf-radext-coa-proxy-05
X-BeenThere: radext@ietf.org
X-Mailman-Version: 2.1.27
Precedence: list
List-Id: RADIUS EXTensions working group discussion list <radext.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/radext>, <mailto:radext-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/radext/>
List-Post: <mailto:radext@ietf.org>
List-Help: <mailto:radext-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/radext>, <mailto:radext-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 15 Aug 2018 18:34:59 -0000

Tim, thanks for your review. Alan, thanks for your responses. I have entered a No Objection ballot.

Alissa

> On Aug 14, 2018, at 4:00 PM, Alan DeKok <aland@deployingradius.com> wrote:
> 
> On Aug 13, 2018, at 8:24 PM, Tim Evens <tievens=40cisco.com@dmarc.ietf.org> wrote:
>> Minor issues:
>> 
>> Nits/editorial comments: 
>> Abstract contains "Section 3.1" which becomes an HTML reference
>> link. This incorrectly links to the current draft section 3.1,
>> not the intended RFC5176 Section 3.1.  This is repeated in 
>> the introduction.  
> 
>  Fixed.
> 
>> IMO, that last sentence would read better with "corrects the omission"
>> instead of "that."
> 
>  It's been reworded to be clearer over all.
> 
>> Code points are not summarized in IANA Considerations section. 
> 
>  Fixed.
> 
>> The references are not formatted per RFC7322.
>> 
>> The HTML rendering of Section 2.2 CoA Processing does not render
>> the RFC5176 link correctly.  Bracketed references normally are followed
>> by some text.
> 
>  I'll fix that.
> 
>> IMO, considering this draft updates 5176, I feel it would be better for the
>> problem statement to be clearer on updates and clarifications. 
> 
>  Perhaps you could suggest text?  My understanding is that the problem statement is fairly clear:
> 
> * CoA proxying is currently impossible
> 
>  And the solution:
> 
> * define a method for doing CoA proxying
> 
>> In section 3.3, while humorous, I suggest dropping "on the planet." 
> 
>  Done.  And clarified substantially as a result of IESG feedback.
> 
>> Section 6 Security Considerations link for Section 11 of RFC6929 is 
>> missing keyword "of." This results in two links instead of the correct
>> link. 
> 
>  Fixed.
> 
> _______________________________________________
> Gen-art mailing list
> Gen-art@ietf.org
> https://www.ietf.org/mailman/listinfo/gen-art