Re: [art] [Technical Errata Reported] RFC7601 (5435)

"John R. Levine" <johnl@iecc.com> Mon, 23 July 2018 01:58 UTC

Date: Sun, 22 Jul 2018 21:58:30 -0400
Message-ID: <alpine.OSX.2.21.1807222152380.18947@ary.qy>
From: "John R. Levine" <johnl@iecc.com>
To: RFC Errata System <rfc-editor@rfc-editor.org>
Cc: Murray Kucherawy <superuser@gmail.com>, ben@nostrum.com, Alexey Melnikov <aamelnikov@fastmail.fm>, adam@nostrum.com, poccil14@gmail.com, art@ietf.org
In-Reply-To: <20180723000558.CD758B80F99@rfc-editor.org>
References: <20180723000558.CD758B80F99@rfc-editor.org>
User-Agent: Alpine 2.21 (OSX 202 2017-01-01)
MIME-Version: 1.0
Content-Type: text/plain; charset="US-ASCII"; format="flowed"
Archived-At: <https://mailarchive.ietf.org/arch/msg/art/GX17KcZ-TaBpzcag_eeg05d4KWU>
Subject: Re: [art] [Technical Errata Reported] RFC7601 (5435)
Precedence: list

I think this erratum is wrong.  The CFWS at the end of propspec is 
actuall in the included pvalue rule and it's optional, so I think the ABNF 
is correct albeit clumsy.  ABNF doesn't have to be LL(1) or anything like 
that although I concur that it would be nice if it is where reasonably 
possible.

Regardless, we are in last call for draft-ietf-dmarc-rfc7601bis in the
dmarc WG so if there's anything to fix, that's the place to fix it.
The ABNF for resinfo and propspec haven't changed.

R's,
John


On Sun, 22 Jul 2018, RFC Errata System wrote:

> The following errata report has been submitted for RFC7601,
> "Message Header Field for Indicating Message Authentication Status".
>
> --------------------------------------
> You may review the report below and at:
> http://www.rfc-editor.org/errata/eid5435
>
> --------------------------------------
> Type: Technical
> Reported by: Peter Occil <poccil14@gmail.com>
>
> Section: 2.2
>
> Original Text
> -------------
> resinfo = [CFWS] ";" methodspec [ CFWS reasonspec ]
>               *( CFWS propspec )
>
>
> Corrected Text
> --------------
> resinfo = [CFWS] ";" methodspec [ CFWS reasonspec ]
>            [ CFWS 1*propspec ]
>
>
> Notes
> -----
> Since propspec includes optional CFWS at end, parsing the current version of resinfo will result in at most one propspec even if methodspec is followed by more than one propspec.
>
> 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.
>
> --------------------------------------
> RFC7601 (draft-ietf-appsawg-rfc7001bis-11)
> --------------------------------------
> Title               : Message Header Field for Indicating Message Authentication Status
> Publication Date    : August 2015
> Author(s)           : M. Kucherawy
> Category            : PROPOSED STANDARD
> Source              : ART Area General Application Working Group
> Area                : Applications and Real-Time
> Stream              : IETF
> Verifying Party     : IESG
>
>

Regards,
John Levine, johnl@iecc.com, Primary Perpetrator of "The Internet for Dummies",
Please consider the environment before reading this e-mail. https://jl.ly

[art] [Technical Errata Reported] RFC7601 (5435) RFC Errata System
Re: [art] [Technical Errata Reported] RFC7601 (54… John R. Levine
Re: [art] [Technical Errata Reported] RFC7601 (54… Peter Occil
Re: [art] [Technical Errata Reported] RFC7601 (54… John R. Levine
Re: [art] [Technical Errata Reported] RFC7601 (54… Peter Occil
Re: [art] [Technical Errata Reported] RFC7601 (54… John R. Levine
Re: [art] [abnf-discuss] [Technical Errata Report… Dave Crocker
[art] BNFs and determinism (Re: [Technical Errata… Carsten Bormann
Re: [art] [Technical Errata Reported] RFC7601 (54… Dale R. Worley
Re: [art] [Technical Errata Reported] RFC7601 (54… Peter Occil