BIND 10 #999: Integrate ACLs into b10-resolver

BIND 10 Development do-not-reply at isc.org
Sat Jun 25 06:59:59 UTC 2011


#999: Integrate ACLs into b10-resolver
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  jinmei
  vorner                             |                Status:  reviewing
                       Type:  task   |             Milestone:
                   Priority:  major  |  Sprint-20110628
                  Component:         |            Resolution:
  Unclassified                       |             Sensitive:  0
                   Keywords:         |           Sub-Project:  DNS
            Defect Severity:  N/A    |  Estimated Difficulty:  4.0
Feature Depending on Ticket:         |           Total Hours:  0
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------

Comment (by jinmei):

 Replying to [comment:12 vorner]:

 > I agree with putting the unification work into a followup ticket, but I
 think it should happen soon, because it will allow adding more types of
 checks easily and it already contains superset of the syntax your loader
 is able to load (for example it can have condition-less actions to modify
 the „default“ result, after merging two tickets currently in review, it
 will support shorter syntax).

 "soon" is fine.  In fact, I already have made some progress locally.
 And, actually, that work made me think completing that part would make
 this branch too big and should better go to a separate ticket.

 And, responding to the build problem first:

 > Anyway, the current code fails to compile for me:
 > {{{
 > run_unittests-ip_check_unittest.o: In function
 `isc::acl::IPCheck<(anonymous namespace)::GeneralAddress>::getAddress()
 const':
 >
 /home/vorner/work/bind10/src/lib/acl/tests/../../../../src/lib/acl/ip_check.h:258:
 undefined reference to `isc::acl::IPCheck<(anonymous
 namespace)::GeneralAddress>::IPV4_SIZE'
 ...
 > }}}

 Hmm, this looks like the imported #998 part, not for the main code of
 this branch.  Does the latest master that has merged with #998
 compiles in your environment?  If so, We should probably merge this
 branch with master.

 BTW, I couldn't reproduce the problem with g++ on my machine or on
 bind10.isc.org.

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


More information about the bind10-tickets mailing list