Re: [tcpm] [Editorial Errata Reported] RFC5925 (5347)

Mirja Kühlewind <mirja.kuehlewind@tik.ee.ethz.ch> Wed, 05 September 2018 19:45 UTC

Return-Path: <mirja.kuehlewind@tik.ee.ethz.ch>
X-Original-To: tcpm@ietfa.amsl.com
Delivered-To: tcpm@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A7996130DCF for <tcpm@ietfa.amsl.com>; Wed, 5 Sep 2018 12:45:15 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.2
X-Spam-Level:
X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3] 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 G-SR1EUYRt-R for <tcpm@ietfa.amsl.com>; Wed, 5 Sep 2018 12:45:10 -0700 (PDT)
Received: from virgo01.ee.ethz.ch (virgo01.ee.ethz.ch [129.132.2.226]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 29E7A130DC7 for <tcpm@ietf.org>; Wed, 5 Sep 2018 12:45:10 -0700 (PDT)
Received: from localhost (localhost [127.0.0.1]) by virgo01.ee.ethz.ch (Postfix) with ESMTP id 425Dgm5PmhzMnrW; Wed, 5 Sep 2018 21:45:08 +0200 (CEST)
X-Virus-Scanned: Debian amavisd-new at virgo01.ee.ethz.ch
Received: from virgo01.ee.ethz.ch ([127.0.0.1]) by localhost (virgo01.ee.ethz.ch [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id s-OwpHI0wuud; Wed, 5 Sep 2018 21:45:07 +0200 (CEST)
X-MtScore: NO score=0
Received: from vpn-global-dhcp2-51.ethz.ch (vpn-global-dhcp2-51.ethz.ch [129.132.209.51]) by virgo01.ee.ethz.ch (Postfix) with ESMTPSA; Wed, 5 Sep 2018 21:45:07 +0200 (CEST)
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 11.5 \(3445.9.1\))
From: Mirja Kühlewind <mirja.kuehlewind@tik.ee.ethz.ch>
In-Reply-To: <4c9ba64bef09362918acfb2d12a65eba@strayalpha.com>
Date: Wed, 05 Sep 2018 21:45:06 +0200
Cc: Ignacio Goyret <ignacio.goyret@nokia.com>, "tcpm@ietf.org Extensions" <tcpm@ietf.org>, touch@isi.edu
Content-Transfer-Encoding: quoted-printable
Message-Id: <DFFDCB90-F0A9-44C0-BE3A-FF74147954B1@tik.ee.ethz.ch>
References: <20180503203252.D2005B82AA4@rfc-editor.org> <fcbc3182691fb2d763d4966b79a48591@strayalpha.com> <201805040027.w440R3ji011041@cliff.eng.ascend.com> <f210b12a-ca8d-4323-05b2-193cceb88669@strayalpha.com> <518c945f-201d-3e20-4406-4135e182c6af@tik.ee.ethz.ch> <4c9ba64bef09362918acfb2d12a65eba@strayalpha.com>
To: Joe Touch <touch@strayalpha.com>
X-Mailer: Apple Mail (2.3445.9.1)
Archived-At: <https://mailarchive.ietf.org/arch/msg/tcpm/GxsdC1zm7uTXiYsqiHWdePM8214>
Subject: Re: [tcpm] [Editorial Errata Reported] RFC5925 (5347)
X-BeenThere: tcpm@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: TCP Maintenance and Minor Extensions Working Group <tcpm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tcpm>, <mailto:tcpm-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tcpm/>
List-Post: <mailto:tcpm@ietf.org>
List-Help: <mailto:tcpm-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tcpm>, <mailto:tcpm-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 05 Sep 2018 19:45:16 -0000

If you could agree, I can update the errata. Thanks!

> Am 05.09.2018 um 21:12 schrieb Joe Touch <touch@strayalpha.com>:
> 
> 
> 
> If we need an errata, we need to agree on the text, which we have not yet.
> 
> 
> 
> Joe
> 
> 
> 
> On 2018-09-05 07:53, Mirja Kühlewind wrote:
> 
>> Hi Joe, hi Ignacio,
>> 
>> is this something that I should set in the "Hold for Update" state?
>> 
>> Mirja
>> 
>> 
>> 
>> On 04.05.2018 03:54, Joe Touch wrote:
>>> 
>>> 
>>> On 5/3/2018 5:20 PM, Ignacio Goyret wrote:
>>>> The reason for the errata is because it happened: someone read
>>>> that text and interpreted that the tcp checksum would only need
>>>> to be cleared when only the TCP-AO option was included because the
>>>> text stated "when included, only the TCP-AO MAC field is zeroed".
>>>> Your suggested text addition does not address this incorrect
>>>> interpretation.
>>> It actually does directly, with this phrase: "within those options".
>>> 
>>>> I also encountered someone that insisted that when only the TCP-AO
>>>> option was included, that it was only the option data, not its header.
>>> That is already addressed in the existing text directly here:
>>> 
>>>    "When TCP options are not included, all TCP options except for TCP-
>>>    AO are omitted from MAC processing."
>>> 
>>> 
>>>> That's why a reference to section 2.2 sounds reasonable to me.
>>> Adding the reference might be useful, but should not be strictly
>>> necessary to interpret the existing text.
>>> 
>>>> IMO, this is *exactly* the reason for an "editorial" errata.
>>>> Technical details are not altered, just language clarifications
>>>> to promote interoperability. I am not suggesting rewriting this or
>>>> any other RFC.
>>> There are many RFCs that have been misinterpreted. That doesn't always
>>> warrant suggested alternate text.
>>> 
>>> Joe
>>> 
>>>> 
>>>> -Ignacio
>>>> 
>>>> 
>>>> At 14:38 5/3/2018, Joe Touch wrote:
>>>> 
>>>>> I do not agree that this is suggested text represents a clarification.
>>>>> 
>>>>> At best, only the one sentence warrants clarification:
>>>>> 
>>>>> From: When included, only the TCP-AO MAC field is zeroed.
>>>>> 
>>>>> To: When TCP options are included, within those options only the TCP-AO MAC field is zeroed.
>>>>> 
>>>>> 
>>>>> However, that context is implicit in the existing text. Yes, we could completely rewrite every RFC to be more clear, but that is not the purpose of errata nor is it productive, IMO.
>>>>> 
>>>>> Joe
>>>>> 
>>>>> On 2018-05-03 16:32, RFC Errata System wrote:
>>>>>> The following errata report has been submitted for RFC5925,
>>>>>> "The TCP Authentication Option".
>>>>>> 
>>>>>> --------------------------------------
>>>>>> You may review the report below and at:
>>>>>> <http://www.rfc-editor.org/errata/eid5347>http://www.rfc-editor.org/errata/eid5347
>>>>>> 
>>>>>> --------------------------------------
>>>>>> Type: Editorial
>>>>>> Reported by: Ignacio Goyret <<mailto:ignacio.goyret@nokia.com>ignacio.goyret@nokia.com>
>>>>>> 
>>>>>> Section: 5.1
>>>>>> 
>>>>>> Original Text
>>>>>> -------------
>>>>>> 3. The TCP header, by default including options, and where the TCP
>>>>>>    checksum and TCP-AO MAC fields are set to zero, all in network-
>>>>>>    byte order.
>>>>>> 
>>>>>>    The TCP option flag of the MKT indicates whether the TCP options
>>>>>>    are included in the MAC.  When included, only the TCP-AO MAC field
>>>>>>    is zeroed.
>>>>>> 
>>>>>>    When TCP options are not included, all TCP options except for TCP-
>>>>>>    AO are omitted from MAC processing.  Again, the TCP-AO MAC field
>>>>>>    is zeroed for the MAC processing.
>>>>>> 
>>>>>> 
>>>>>> Corrected Text
>>>>>> --------------
>>>>>> 3. The TCP header and TCP options, where the TCP checksum and TCP-AO
>>>>>>    MAC fields are always set to zero, all in network-byte order.
>>>>>> 
>>>>>>    The TCP option flag of the MKT indicates which TCP options are
>>>>>>    included in the MAC. When TCP options are not included, only the
>>>>>>    TCP option for TCP-AO (as described in Section 2.2) is included
>>>>>>    in the MAC. Otherwise, all the TCP options are included in the MAC.
>>>>>> 
>>>>>> 
>>>>>> Notes
>>>>>> -----
>>>>>> Rewording for clarity and simplification.
>>>>>> The original text could lead to confusion re '...When included, only the TCP-AO MAC field is zeroed.'
>>>>>> 
>>>>>> Instructions:
>>>>>> -------------
>>>>>> This erratum is currently posted as "Reported". If necessary, please
>>>>>> use "Reply All" to discuss whether it should be verified or
>>>>>> rejected. When a decision is reached, the verifying party
>>>>>> can log in to change the status and edit the report, if necessary.
>>>>>> 
>>>>>> --------------------------------------
>>>>>> RFC5925 (draft-ietf-tcpm-tcp-auth-opt-11)
>>>>>> --------------------------------------
>>>>>> Title               : The TCP Authentication Option
>>>>>> Publication Date    : June 2010
>>>>>> Author(s)           : J. Touch, A. Mankin, R. Bonica
>>>>>> Category            : PROPOSED STANDARD
>>>>>> Source              : TCP Maintenance and Minor Extensions
>>>>>> Area                : Transport
>>>>>> Stream              : IETF
>>>>>> Verifying Party     : IESG
>>> 
>>> _______________________________________________
>>> tcpm mailing list
>>> tcpm@ietf.org
>>> https://www.ietf.org/mailman/listinfo/tcpm
>>> 
>> 
>> _______________________________________________
>> tcpm mailing list
>> tcpm@ietf.org
>> https://www.ietf.org/mailman/listinfo/tcpm