BIND 10 #1244: NSEC support in isc::auth::Query

BIND 10 Development do-not-reply at isc.org
Mon Dec 5 21:43:30 UTC 2011


#1244: NSEC support in isc::auth::Query
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  UnAssigned
  vorner                             |                Status:  assigned
                       Type:  task   |             Milestone:
                   Priority:  major  |  Sprint-20111206
                  Component:         |            Resolution:
  b10-auth                           |             Sensitive:  0
                   Keywords:         |           Sub-Project:  DNS
            Defect Severity:  N/A    |  Estimated Difficulty:  1
Feature Depending on Ticket:         |           Total Hours:  0
  Datasrc refactoring                |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------

Comment (by jinmei):

 Assuming #1308 is mostly ready, I think we can also close this
 meta ticket.

 I've looked at the resulting code by merging trac1308 to master,
 and haven't found a point that require obvious refactoring (e.g. due
 to code duplicate).  Some of the addXXXProof variants share common
 logic, but each is slightly different from others, and since they are
 basically quite compact, I think it's okay to leave them as they are.

 I've made a few trivial cleanups and for that purpose created a
 separate branch (trac1244).  One, possibly not so trivial, cleanup is
 the change of the method name: from addWildcardNxrrsetProof to
 addWildcardNXRRSETProof.  I did it so that it would be consistent
 with addNXDOMAINProof (character case policy should better consistent
 between NXDOMAIN and NXRRSET, and since they are kind of acronyms I
 think all-upper-case is a better choice).

 I've put this ticket to the review queue just in case someone notices
 something wrong, but since it's quite trivial I basically plan to
 merge the cleanup changes and close this ticket (after #1308 is
 closed) unless I hear something soon.

-- 
Ticket URL: <http://bind10.isc.org/ticket/1244#comment:10>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development


More information about the bind10-tickets mailing list