> >       dmarc-record    = dmarc-version dmarc-sep *(dmarc-tag dmarc-sep)
>
> No, in April we said something like so:
>
>          dmarc-record    = dmarc-version *(dmarc-sep dmarc-tag) [dmarc-sep]

Indeed; the difference between "must end with dmarc-sep" and "may end
with dmarc-sep".  I also recall us choosing the latter.

> >       dmarc-version   = "v" *WSP "=" *WSP %x44 %x4d %x41 %x52 %x43 %x31
>
> Can we use RFC 7405?  It is more readable:
>
>          dmarc-version   = "v" *WSP "=" *WSP %s"DMARC1" ; case sensitive

Strong agreement on readability here.

Another point on readability:

> >       dmarc-request   = "p" *WSP "=" *WSP
> >                         ( "none" / "quarantine" / "reject" )

Quite a few constructs include this: *WSP "=" *WSP

Is it worth creating something like this:

equals = *WSP "=" *WSP

...so we could then use...

dmarc-request   = "p" equals ( "none" / "quarantine" / "reject" )

...and so on?  I find that to be easier on the eyes, though others
might see it as adding complication to the ABNF.  Thoughts?

Barry

_______________________________________________
dmarc mailing list
dmarc@ietf.org
https://www.ietf.org/mailman/listinfo/dmarc

Reply via email to