At Wed, 13 Jan 2016 13:53:06 -0800,
"Paul Hoffman" <paul.hoff...@vpnc.org> wrote:

> We'd really like folks to review it, particularly because this is such a
> large change from earlier versions. The document is still definitely
> open for discussion, and if there is consensus to reverse some of the
> changes, that's of course fine. Regardless, let's get this published in
> a form where everyone (?) is happy.

Here are my comments on the 06 version:

- Section 3.1

   [...]  This would be when the resolver starts with an empty cache,
   and when one or more of the NS records for the root servers has
   expired.

  This seems to talk about a recursive server behavior that allows a
  subset of an RRset to expire.  Is that the intent?  Is there any
  reason why we can't simply say "...and when the NS RRSet for the
  root zone has expired."? (btw I think 'root zone' is better
  than 'root servers' here).

- Section 3.1: some modern recursive servers "prefetch" un-expired
  RRsets.  We might also mention that case here.

- Section 3.3

   [...]  At the time this
   document is being published, there is little use to performing DNSSEC
   validation on the priming query because the "root-servers.net" zone
   is not signed, and so a man-in-the-middle attack on the priming query
   can result in malicious data in the responses.

  I think this text needs more explanation.  Since the priming query
  is "./NS", it's not immediately obvious why the fact that
  "root-servers.net" is unsigned matters.  This text seems to be based
  on some implicit assumptions/facts such as:
  - all root servers are authoritative for the root-servers.net zone
  - we'll eventually need AAAA and A RRsets of the NS names
  - even if ./NS is signed, it's not very useful in practice unless
    these AAAA and A are also signed
  and, reading between the lines I think I can agree with the seeming
  intent, but IMO it should be explained more clearly and explicitly.

- Section 4.1

   answer section) and an Additional section with A and/or AAAA RRSets
   for the root name servers pointed at by the NS RRSet.

  Similar to the previous point, it seems to be based on some implicit
  assumptions including:
  - all root servers are authoritative for the root-servers.net zone
  - all these servers populate the additional section from a different
    zone they are authoritative for than that for the query name ("."
    in this case)
  - none of the root server implementations use the
    "minimal-responses" (or equivalent) option

  I think these should be clearly stated.

- Section 4.2

   For an EDNS response, a resolver SHOULD consider the address
   information found in the Additional section complete for any
   particular server that appears at all.

  This one also seems to assume some particular behavior of the root
  server implementations.  At least protocol-wise, we cannot
  necessarily assume this completeness, right?  Then I'd also suggest
  clarifying the assumption.

Editorial nits:

- Section 3.2: s/other other/other/

   [...]  Two other other
   common methods include picking the first from the list, and

- Section 4.2: s/to to/to/

   [...]  Instead, the recursive resolver needs to to
   issue direct queries for A and AAAA RRSets for the remaining names.

--
JINMEI, Tatuya

_______________________________________________
DNSOP mailing list
DNSOP@ietf.org
https://www.ietf.org/mailman/listinfo/dnsop

Reply via email to