Re: [dmarc-ietf] I-D Action: draft-ietf-dmarc-dmarcbis-07.txt

[Scott Kitterman <[e-mail]>]

On Friday, April 29, 2022 3:29:33 AM EDT Robert wrote:
> A couple of little issues in the ABNF of the formal definition section 5.4:
> 
>      dmarc-psd       = "psd" *WSP "=" ( "y" / "n" )
> Should be:
>      dmarc-psd       = "psd" *WSP "=" *WSP ( "y" / "n" / "u" )
> 
> and
>      dmarc-test      = "t" *WSP "=" ( "y" / "n" )
> should be:
>      dmarc-test      = "t" *WSP "=" *WSP ( "y" / "n" )
> 
> Also I've noticed that the dmarc-uri definition seems to be missing
> the stuff about the file size limitation? At the moment it's just:
>      dmarc-uri       = URI
>                        ; "URI" is imported from [RFC3986]; commas (ASCII
>                        ; 0x2C) and exclamation points (ASCII 0x21)
>                        ; MUST be encoded
> not sure what it should be.

I'm not any kind of an ABNF expert, but adding the missing "u" for dmarc-psd 
is definitely correct and I believe the *WSP for dmarc-psd and dmarc-test are 
correct also.

I would suggest those go into the next revision.  Any suggestions on the 
dmarc-uri question?

Scott K