On 1/7/25 15:31, Todd Herr wrote:
> This version contains some minor cleanup from a couple of pull requests
> and some on-list discussion.
Thanks for pulling my PR, and incorporating fixes for the IANA issues I
noted.
I don't know how you work on dmarcbis or how often you read the list
email, but as a new contributor trying to find my way around here I
would appreciate it if you or someone would reply to those messages
acknowledging or otherwise discussing the issue. Maybe just saying that
it will be fixed in the next revision, or requesting I make a PR if you
agree with the proposed text change, if any.
That way I get some feedback indicating if what I do is valuable or if I
should focus elsewhere.
This time I found out that it was indeed valuable, actionable input by
reading the updated draft where the changes had been incorporated.
This was just a self serving observation, I do not want to start a wider
discussion about this. Feel free to disregard it entirely if it does not
fit your workflow.
However, the ABNF for dmarc-urilist is now botched, and too long causing
a line length warning.
The ABNF for dmarc-urilist I submitted as part of the obs-dmarc-uri
addition was changed to:
dmarc-urilist = dmarc-uri / obs-dmarc-uri *(*WSP "," *WSP dmarc-uri /
obs-dmarc-uri)
Which means that a valid value is one of:
1) dmarc-uri
2) obs-dmarc-uri *(*WSP "," *WSP dmarc-uri / obs-dmarc-uri)
I think we need to go back to a variation of my original version that
had properly structured alternatives:
dmarc-urilist = ( dmarc-uri *(*WSP "," *WSP dmarc-uri) )
/ ( obs-dmarc-uri *(*WSP "," *WSP obs-dmarc-uri) )
The outer sequence group for each of the alternatives are not strictly
needed, but RFC5234, Section 3.5 "strongly advise" that they are added,
to "avoid misinterpretation by casual readers".
I think I see why you changed it, it is unreasonable to require all
URI's in the list either having a size part or no size part. Mixed use
should be supported, as in the following updated ABNF:
dmarc-urilist = (dmarc-uri / obs-dmarc-uri)
*(*WSP "," *WSP (dmarc-uri / obs-dmarc-uri))
Daniel K.
_______________________________________________
dmarc mailing list -- [email protected]
To unsubscribe send an email to [email protected]
