Last Call Review of draft-ietf-add-svcb-dns-06
review-ietf-add-svcb-dns-06-secdir-lc-salowey-2022-07-08-00

I have reviewed this document as part of the security directorate's
ongoing effort to review all IETF documents being processed by the
IESG.  These comments were written primarily for the benefit of the
security area directors.  Document editors and WG chairs should treat
these comments just like any other last call comments.

The summary of the review is the document is ready.

A few comments:

- Section 8.1.2 - good description of this problem, it seems like some of this
should have been discussed in the doh document, but I couldn't find any.  If
there is relevant considerations in the doh document then you should reference
them here.  It seems that the recommendation "To mitigate redirection attacks,
a client of this SVCB mapping MUST NOT identify or authenticate itself when
performing DNS queries, except to servers that it specifically knows are not
vulnerable to such attacks." would be difficult to implement since its not
clear how the client gets this information and really should be a consideration
for the server implementations/deployments that require authentication.  I'm
not really sure what to do about this except as a consideration for a revision
of DoH.