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