Re: [Gen-art] Genart telechat review of draft-ietf-kitten-rfc5653bis-06

"Joel M. Halpern" <jmh@joelhalpern.com> Wed, 03 January 2018 02:50 UTC

Return-Path: <jmh@joelhalpern.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 64468127444; Tue, 2 Jan 2018 18:50:24 -0800 (PST)
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=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=joelhalpern.com
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 eviVLmaf8djq; Tue, 2 Jan 2018 18:50:21 -0800 (PST)
Received: from maila2.tigertech.net (maila2.tigertech.net [208.80.4.152]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 59B2812426E; Tue, 2 Jan 2018 18:50:21 -0800 (PST)
Received: from localhost (localhost [127.0.0.1]) by maila2.tigertech.net (Postfix) with ESMTP id 42933A0A429; Tue, 2 Jan 2018 18:50:21 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=joelhalpern.com; s=1.tigertech; t=1514947821; bh=lxyeSIH5+RKWXTlbULa2hDlGmPi0R1TMddmj7/yzaMs=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=mFOIbPK84tlQELSQaBsANSH7uLK9XPzEDSnoPEzYpYGmY83ipqsl4vbrTmydlymrW iIy2MrVvOSvWNBL1CfpFP4Ma4rb5ev6+q4z/G2CNLTUoz8kzUszmW9Zm7BTnABX0ZJ JrtRf9ZVvSePuCT4MOtBBKuLH4zRjzOopzR/kYjY=
X-Virus-Scanned: Debian amavisd-new at maila2.tigertech.net
Received: from Joels-MacBook-Pro.local (unknown [50.225.209.67]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by maila2.tigertech.net (Postfix) with ESMTPSA id EE679A0A426; Tue, 2 Jan 2018 18:50:19 -0800 (PST)
To: Weijun Wang <weijun.wang@oracle.com>
Cc: Benjamin Kaduk <kaduk@mit.edu>, gen-art@ietf.org, kitten@ietf.org, ietf@ietf.org, draft-ietf-kitten-rfc5653bis.all@ietf.org
References: <151493583156.30989.1076207750886953383@ietfa.amsl.com> <20180103013808.GG50827@kduck.kaduk.org> <25d5a1bb-e7bb-431b-6632-09904a581d77@joelhalpern.com> <074DA813-1E7F-4C03-AEEE-5D76E8804C31@oracle.com>
From: "Joel M. Halpern" <jmh@joelhalpern.com>
Message-ID: <41bbbe7d-0f35-78ad-a5cd-673488f3ac09@joelhalpern.com>
Date: Tue, 02 Jan 2018 21:50:19 -0500
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0) Gecko/20100101 Thunderbird/52.5.0
MIME-Version: 1.0
In-Reply-To: <074DA813-1E7F-4C03-AEEE-5D76E8804C31@oracle.com>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/SYV1GOoAXwmAMFge844MiYCM9I8>
Subject: Re: [Gen-art] Genart telechat review of draft-ietf-kitten-rfc5653bis-06
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.22
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: <https://mailarchive.ietf.org/arch/browse/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, 03 Jan 2018 02:50:24 -0000

Personal opinion:
Given that to my reading "must" is used on the document in both the RFC 
2119 sense and in a conventional English language sense, it would be 
worth clarifying the intention.  As such, I think it would be better to 
use the 8174 reference and go through changing the right ones to upper case.

Yours,
Joel

On 1/2/18 9:40 PM, Weijun Wang wrote:
> Hi Joel and Ben
> 
> Author here.
> 
> I think I've removed the section because in fact none of the keywords appears as capitalized inside the original document. In fact, RFC 8174 has "that only UPPERCASE usage of the key words have the defined special meanings".
> 
> I assume I'll need to go through the document and make some UPPERCASE and some not, depending on the actual meanings.
> 
> Or, since this is a bis and changing the cases would be considered an re-intepretation of the whole document (which wasn't my goal), is it more reasonable to keep using RFC 2119 and leave all "must" and "required" in lowercase?
> 
> Thanks
> Weijun
> 
> 
>> On Jan 3, 2018, at 9:49 AM, Joel M. Halpern <jmh@joelhalpern.com> wrote:
>>
>> Thanks Ben.  That would be good.
>> Yours,
>> Joel
>>
>> On 1/2/18 8:38 PM, Benjamin Kaduk wrote:
>>> Hi Joel,
>>> On Tue, Jan 02, 2018 at 03:30:31PM -0800, Joel Halpern wrote:
>>>> Reviewer: Joel Halpern
>>>> Review result: Ready with Issues
>>>>
>>>> I am the assigned Gen-ART reviewer for this draft. The General Area
>>>> Review Team (Gen-ART) reviews all IETF documents being processed
>>>> by the IESG for the IETF Chair. Please wait for direction from your
>>>> document shepherd or AD before posting a new version of the draft.
>>>>
>>>> For more information, please see the FAQ at
>>>>
>>>> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
>>>>
>>>> Document: draft-ietf-kitten-rfc5653bis-06
>>>> Reviewer: Joel Halpern
>>>> Review Date: 2018-01-02
>>>> IETF LC End Date: 2017-09-11
>>>> IESG Telechat date: 2018-01-25
>>>>
>>>> Summary: This document is ready for publication as a Proposed Standard RFC
>>>>
>>>> Major issues: None
>>>>
>>>> Minor issues:
>>>>      Although ID-Nits does not complain about it, I can find no reference to
>>>>      RFCs 2119 or 8174.  Some of the uses of "must" int he document are along
>>>>      the lines of "inherently follows", which is not normative language.  But
>>>>      other uses are clearly normative in structure.   It is unclear why the
>>>>      reference to RFC 2119 was removed as part of this update.
>>> Thanks for the review -- I'm a bit surprised that id-nits does not
>>> complain about the omission.
>>> I do not know why the -00 dropped that clause, but it does seem like
>>> the current normal text citing 8174 should be added before
>>> publication.
>>> -Ben
> 
>