Skip to main content

Last Call Review of draft-ietf-dmarc-dmarcbis-36
review-ietf-dmarc-dmarcbis-36-dnsdir-lc-gieben-2024-12-08-00

Request Review of draft-ietf-dmarc-dmarcbis
Requested revision No specific revision (document currently at 38)
Type Last Call Review
Team DNS Directorate (dnsdir)
Deadline 2024-12-04
Requested 2024-11-20
Authors Todd Herr , John R. Levine
I-D last updated 2024-12-08
Completed reviews Genart Last Call review of -36 by Ines Robles (diff)
Artart Last Call review of -36 by Scott Hollenbeck (diff)
Secdir Last Call review of -36 by Stephen Farrell (diff)
Dnsdir Last Call review of -36 by R. (Miek) Gieben (diff)
Dnsdir Telechat review of -38 by R. (Miek) Gieben
Assignment Reviewer R. (Miek) Gieben
State Completed
Request Last Call review on draft-ietf-dmarc-dmarcbis by DNS Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/dnsdir/pUfmmlSIF0-WgdCNDMjVBzRDRsU
Reviewed revision 36 (document currently at 38)
Result Ready w/nits
Completed 2024-12-08
review-ietf-dmarc-dmarcbis-36-dnsdir-lc-gieben-2024-12-08-00
Hello,

I've just reviewed version 36 of this draft. Found some nits, and I really have
only one question.

In section 4.5 RFC 1035 is referenced on how a TXT record is structured.
However another question I got is how to deal with multiple _dmarc TXT records?
Is that allowed or is the assumption there is only one? (Admittedly I haven't
read the normative references, so it may be explained there, if so I'm happy to
be told, "Miek, go read the references.")

Also section 4.5:

   DMARC's use of the Domain Name Service   => Domain Name System

In section 4.10, there is this text:

   If x < 8, remove the left-most (highest-numbered) label from the subject
   domain. If x >= 8, remove the left-most (highest-numbered) labels from the
   subject domain until 7 labels remain.

It took me quite a while to realize that '...until 7 labels remain' is the key
difference here. This may benefit from some slight rewwording?

Typos:

* section 4.10: discovry - discovery
* section 5.1.8: [Author Domain]{#author-domain) -> [Author
domain](#author-domain)