BIND 10 #1414: Zonemgr/secondary_zones[]/class required

[BIND 10 Development]

#1414: Zonemgr/secondary_zones[]/class required
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  jinmei
  jinmei                             |                Status:  reviewing
                       Type:         |             Milestone:
  defect                             |  Sprint-20111220
                   Priority:  major  |            Resolution:
                  Component:         |             Sensitive:  0
  configuration                      |           Sub-Project:  Core
                   Keywords:         |  Estimated Difficulty:  4
            Defect Severity:  N/A    |           Total Hours:  4
Feature Depending on Ticket:  none   |
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by jelte):

 * owner:  jelte => jinmei


Comment:

 Replying to [comment:12 jinmei]:
 >
 > >
 > > Might as well fix it now, added to-and-from conversion, with exception
 catch to provide a slightly better error (and to convert to one
 exception). Added a few tests as well (but not extensively for every
 possible bad dname)
 >
 > Looks okay, but to be even more perfect, I guess we need to do:
 > - perform syntax check on the class, too (reject "BADCLASS", etc)
 > - confirm whether the comparison is case-insensitive ('example.COM/in'
 >   should be treated as 'example.com./IN'.)
 >

 ok, done

 updated changelog (added "and class")

 [bug] jelte
 Fixed a bug where adding Zonemgr/secondary_zones without explicitely
 setting the class value of the added zone resulted in a cryptic error in
 bindctl ("Error: class"). It will now correctly default to IN if not set.
 This also adds better checks on the name and class values, and better
 errors if they are bad.

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