Skip to main content

Last Call Review of draft-ietf-anima-brski-prm-17
review-ietf-anima-brski-prm-17-dnsdir-lc-tale-2025-02-04-00

Request Review of draft-ietf-anima-brski-prm
Requested revision No specific revision (document currently at 18)
Type Last Call Review
Team DNS Directorate (dnsdir)
Deadline 2025-01-30
Requested 2025-01-16
Authors Steffen Fries , Thomas Werner , Eliot Lear , Michael Richardson
I-D last updated 2025-02-04
Completed reviews Secdir Early review of -10 by Charlie Kaufman (diff)
Secdir Early review of -05 by Charlie Kaufman (diff)
Yangdoctors Early review of -05 by Martin Björklund (diff)
Iotdir Early review of -05 by Marco Tiloca (diff)
Iotdir Last Call review of -15 by Marco Tiloca (diff)
Secdir Last Call review of -17 by Wes Hardaker (diff)
Opsdir Last Call review of -15 by Ran Chen (diff)
Dnsdir Last Call review of -17 by David C Lawrence (diff)
Genart Last Call review of -17 by Paul Kyzivat (diff)
Assignment Reviewer David C Lawrence
State Completed
Request Last Call review on draft-ietf-anima-brski-prm by DNS Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/dnsdir/hjUZpUBixSUOa_VBeyitx7VxwW8
Reviewed revision 17 (document currently at 18)
Result Ready w/nits
Completed 2025-02-04
review-ietf-anima-brski-prm-17-dnsdir-lc-tale-2025-02-04-00
For the DNS Directorate, I have focused my review on the DNS-salient
aspects of the draft.

The core protocol use of DNS-SD looks spot-on, so no issues there.

This draft does a callout to RFC 8995 for core definitions, which does
fortunately include a definition of "Domain" that makes it clear that
it isn't talking only about a domain name in the common DNS sense.
However, neither 8995's Terminology section nor the one here define
Domain Registrar, and given that it is a reasonably common term on the
Internet for which people will have a preconceived notion of its
meaning, it'd be good to identify in Terminology here not only what it
does mean, but to explicitly acknowledge that it is not a DNS
registrar.

Section 10.2 for IANA is titled "DNS Service Names" but fails to
mention where IANA is registering this information.  I believe you
mean the Service Name and Transport Protocol Port Number Registry,
which is not a DNS Service Name registry.  Please add the registry
name in the section, and delete DNS from the section title.

I've noted that a number of other reviews have already covered matters
of proofreading, and rather than attempt to de-duplicate the minor
errors I also saw, I've instead trusted that they were all already
covered.