Re: [Idr] AD Review of draft-ietf-idr-ext-opt-param-09

Jeffrey Haas <jhaas@pfrc.org> Mon, 22 March 2021 21:26 UTC

Return-Path: <jhaas@pfrc.org>
X-Original-To: idr@ietfa.amsl.com
Delivered-To: idr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 92E0E3A1296; Mon, 22 Mar 2021 14:26:44 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0
X-Spam-Level:
X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[HTML_MESSAGE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
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 sdSr7-JOZcqB; Mon, 22 Mar 2021 14:26:43 -0700 (PDT)
Received: from slice.pfrc.org (slice.pfrc.org [67.207.130.108]) by ietfa.amsl.com (Postfix) with ESMTP id DF3CE3A128F; Mon, 22 Mar 2021 14:26:42 -0700 (PDT)
Received: from dresden.attlocal.net (99-59-193-67.lightspeed.livnmi.sbcglobal.net [99.59.193.67]) by slice.pfrc.org (Postfix) with ESMTPSA id 927AE1E409; Mon, 22 Mar 2021 17:48:22 -0400 (EDT)
Content-Type: multipart/alternative; boundary="Apple-Mail=_02239ACA-A46A-496B-98F3-44AB785287D2"
Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.120.23.2.4\))
From: Jeffrey Haas <jhaas@pfrc.org>
In-Reply-To: <CAMMESszRZKtnW-qW6kSjn5njgQayZdtNSbL=YJr8VAKVu2XX4A@mail.gmail.com>
Date: Mon, 22 Mar 2021 17:26:40 -0400
Cc: John Scudder <jgs@juniper.net>, "draft-ietf-idr-ext-opt-param@ietf.org" <draft-ietf-idr-ext-opt-param@ietf.org>, IDR List <idr@ietf.org>, Sue Hares <shares@ndzh.com>, "idr-chairs@ietf.org" <idr-chairs@ietf.org>
Message-Id: <BCCF4743-B670-4A7E-890F-DA7FB0EDFC20@pfrc.org>
References: <CAMMESsw3eVnfiJ8RQ4GSSzp4b1T2n6hm-nSZ6xuyK9XCMb1pAg@mail.gmail.com> <862CFB7E-DB91-49DE-87B7-B68F94BBA406@juniper.net> <CAMMESszRZKtnW-qW6kSjn5njgQayZdtNSbL=YJr8VAKVu2XX4A@mail.gmail.com>
To: Alvaro Retana <aretana.ietf@gmail.com>
X-Mailer: Apple Mail (2.3608.120.23.2.4)
Archived-At: <https://mailarchive.ietf.org/arch/msg/idr/JTDln9m6osQIQzBbU81Wgx9vcNQ>
Subject: Re: [Idr] AD Review of draft-ietf-idr-ext-opt-param-09
X-BeenThere: idr@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Inter-Domain Routing <idr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/idr>, <mailto:idr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/idr/>
List-Post: <mailto:idr@ietf.org>
List-Help: <mailto:idr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/idr>, <mailto:idr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 22 Mar 2021 21:26:45 -0000


> On Mar 22, 2021, at 5:15 PM, Alvaro Retana <aretana.ietf@gmail.com> wrote:
> 
>>> Besides clarifying that, I think that new OPEN Message Error subcodes
>>> are needed for the cases where the length is invalid. This type of
>>> error is possible in the non-extended version of the OPEN too -- I
>>> guess rfc4271 just assumed that this error would never happen.
>> 
>> RFC 4271 left a lot to be desired in terms of nuanced error codes. I agree
>> that we could introduce a subcode called something like “optional parameter
>> field length mismatch” to be sent if the optional parameters length exceeds
>> the unconsumed OPEN length. However, I’m disinclined to do it in this
>> document, because it’s a pre-existing problem, it exists with the
>> non-extended encoding as well, so any fix for it should really fix both
>> cases. As such, it seems a funny fit for this doc.
> 
> As I said in my reply to Jeff, I'm ok with that not being included in
> this document.

And I wouldn't push for it.  Which means there's probably no chairs' conversation to be had. :-)

-- Jeff