BIND 10 #2911: don't fallback from ixfr to axfr due to data source error

BIND 10 Development do-not-reply at isc.org
Thu May 30 08:25:10 UTC 2013


#2911: don't fallback from ixfr to axfr due to data source error
-------------------------------------+-------------------------------------
            Reporter:  jinmei        |                        Owner:
                Type:  defect        |  jinmei
            Priority:  medium        |                       Status:
           Component:  xfrin         |  reviewing
            Keywords:                |                    Milestone:
           Sensitive:  0             |  Sprint-20130611
         Sub-Project:  DNS           |                   Resolution:
Estimated Difficulty:  4             |                 CVSS Scoring:
         Total Hours:  3.22          |              Defect Severity:  N/A
                                     |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
Changes (by vorner):

 * owner:  vorner => jinmei
 * totalhours:  0 => 3.22


Comment:

 Hello

 Replying to [comment:19 jinmei]:
 > > Yes, unless it's a lot of work, as it's only a minor detail.
 >
 > Hmm, it's not clear this "yes" means I should update it or not.  Can
 > you clarify?

 Yes, please update it (possibly at merge time).

 > > No. I mean the fact that we create a new zone if there wasn't one
 before in the database. And it's not inside the transaction, so if the
 transfer fails for some reason, the data is not committed, but the new
 zone is still left in the database, empty.
 >
 > Ah, okay, that's right.  I'm inclined to say it's out of scope for
 > this ticket, as it's not something introduced by this branch and we
 > are also expecting fundamental rewrite of xfr's with zone management
 > framework.  But if you think we should update it within this task
 > strongly, I'm okay with doing it too.

 No, I agree it's out of scope of this ticket. I just asked to make sure
 the branch didn't fix that as a side effect.


 I believe this can be now merged.

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


More information about the bind10-tickets mailing list