I followed previous discussions on this draft but don't remember all of
the details, so I may be rehashing some old discussions (in which case
that's not intentional...)

Overall: I think the idea is useful and it's eventually worth
publishing.  But I've noticed a few non-trivial points that may have
to be addressed before publication.

- Section 1 (introduction), the first paragraph:

   This document specifies how a child zone in the DNS ([RFC1034],
   [RFC1035]) can publish a record to indicate to a parental agent that
   it may copy and process certain records from the child zone.  The
   existence of and value change of the record may be monitored by a
   parental agent and acted on as appropriate.

 I vaguely remember someone already pointed this out, but anyway: I'm
 afraid the term "parental agent" is not so widely shared that we can
 safely use it without first giving the definition.  One easy way to
 address this would be to add a forward reference to Section 1.1 at
 the first occurrence of the term.  If possible, it would be even
 nicer if we can avoid using this term until the definition is given
 in Section 1.1.  For the same reason, it would be safer to avoid
 using it in the abstract.

- Section 3 (in general)
  Do we need some way to avoid making the parental agent keep fetching
  RRsets specified in CSYNC only to confirm they are still the latest?
  draft-ietf-dnsop-delegation-trust-maintainance has a way to avoid
  that by removing CDS/CDNSKEY once all parent name servers are
  updated.

- In Section 3.1, it suggests a sequence of CSYNC and other followup
  queries enclosed by SOA queries, and requires serials of the SOAs be
  identical (with a MUST).  I wonder if this is reliable enough for a
  rapidly changing zone, such as those accepting dynamic updates at a
  very high rate.  We might say such cases are out of scope of this
  mechanism, but I personally think such an environment is not so
  deviant that a standard-track protocol can casually ignore.  At the
  very least I would like to see some explicit consideration text on
  the expected limitation (if any) regarding this point

- Section 3.1

   [...]  If
   state is being kept by the parental agent and the SOA serial number
   is less than the last time a CSYNC record was processed, this CSYNC
   record SHOULD NOT be processed.  Similarly, if state is being kept by
   the parental agent and the SOA Serial Field of the CSYNC record is
   less than the SOA Serial Field of the CSYNC record from last time,
   then this CSYNC record SHOULD NOT be processed.

  I'm not sure about the point of these "SHOULD NOT"s.  If it's okay
  with ignoring mismatches with stored state, why would the parental
  agent bother to keep the state in the first place?  Since keeping
  the state itself is optional, it seems to make more sense to use
  "MUST NOT" here.

- Section 3.2

   NS records found within the child's zone should be copied verbatim
   and the result published within the parent zone should be an exact
   matching set of NS records.

  Does "verbatim" indicate that the TTL should also be copied?  The
  same question applies to Section 3.2.2, although "verbatim" isn't
  used in that section.

- Section 3.2: what if the followup NS query results in 'no data'?  Of
  course, this means the child zone is broken, but if the parent also
  removes the NS RRsets, subsequent resolution for the zone will
  immediately fail at the parent zone; on the other hand, if the
  parent just ignores such result and keeps the NS RRset (and if it's
  actually still usable), subsequent resolution will still somehow
  work in many cases in practice.  I don't know if that's the desired
  scenario, and we might rather make it fail sooner rather than
  leaving the half-broken state longer.  In any case, I think it would
  be nicer to mention this case (and what the parent should do) in
  this document.

- Section 4.2
  We may want to be clearer about how the child name servers and their
  addresses are determined to send CSYNC queries if they are not
  manually configured.  That is, this should essentially come from
  the NS and AAAA/A records at the parent zone, and some of these may
  be obsolete or even unusable at the time of query (in fact,
  reflecting such changes is exactly the purpose of these queries).
  This also means the child cannot simply update all NS (or AAAA or A)
  records at once, making the old ones unworkable, and expect the
  parent will catch up with it.  This may be obvious in some sense,
  but may still be worth noting.

- Section 4.3

   Children deploying NS records pointing to domain-names within their
   own children (the "grandchildren") SHOULD ensure the grandchildren's
   associated glue records are properly set before publishing the CSYNC
   record.  I.e., it is imperative that proper communication and
   synchronization exist between the child and the grandchild.

  I'm afraid this setup requires more discussion.  In the following
  configuration:
  parent: example.com.
  child: child.example.com.
    child.example.com. NS ns.grand.child.example.com.
    grand.child.example.com. NS ns.grand.child.example.com.
    ns.grand.child.example.com. AAAA 2001:db8::1
  grand child: grand.child.example.com.
    ns.grand.child.example.com. AAAA 2001:db8::1
  If the AAAA record is changed, the child will update its CSYNC record
  with setting the bit for AAAA.  According to Section 3.1, the parental
  agent will send a query for the AAAA record to the child's name
  server, but it will return a delegation to the grandchild, not the
  requested AAAA itself, let alone its RRSIG.  The parental agent
  could then resolve and verify the AAAA separately, but it breaks the
  "atomicity" of the operation that this section seems to seek by
  enclosing the whole set of queries with two SOA queries.

- Section 6: unfortunately code 61 was already registered for OPENPGPKEY.
   [ To be removed prior to publication: The CDS (59), CDNSKEY (60) and
   the CSYNC records are all conceptually similar - if the code-point 61
   happens to still be Unassigned when the IANA processes this, it would
   be nice if that could be used for this.]

- Editorial nits
  - Section 2: s/these/three/
   The CSYNC RRType contains, in its RDATA component, these parts: an
  - Section 2: s/Section Section/Section/ (there are several instances
    of this error)
   data is processed is described in Section Section 3.
  - Section 2: s/any anything/anything/ (?)
   if any of the validation results indicate any anything other than

--
JINMEI, Tatuya

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

Reply via email to