[master] Merge branch 'trac1626'

Changelog for #1715

Merge #1715

Crash of bindctl when the config unset command was invoked. The code of
the command was missing, probably never implemented and obviously never
tested.

[1746] removed SQLite3Accessor::getRecordDiff, and adjusted the test as such.

it was intended to be a short term hack until we implement getDiffs(),
but apparently we forgot to complete the cleanup task.  So this is just an
originally planned cleanup.
note that there was a bug in the test code.  The change to diff_add_a_data
is not to hide a problem in the tested code, but it's a fix to the test's bug.

[1820] use the ScopedSocket wrapper throughout the code for more safety.

[1820] avoid using the removed DNSService interfaces.

also made other cleanups, making it the code more exception safe.

[1820] removed DNSService::addServer() as they were mostly unused.

Likewise, some of the constructors of DNSService were removed too.
the test code specific to the removed interfaces were also removed.

[master] indentation fix.

[1773] comment updates due to interface changes as part of fix.

[1773new] [1773] initial fix for clang++/Lion build failure

[1773] initial fix for clang++/Lion build failure

[master] Merge branch 'trac1535'

[1535] Merge branch 'trac1535' of ssh://git.bind10.isc.org/var/bind10/git/bind10 into trac1535

[324] use a much larger major version (1000) for "newschema" test DBs.

it will reduce the risk of getting conflict soon due to schema updates.

[324new2] renamed major/minor version variables so they are more consistent.

[324] make sure initial SQLite3 DB creation is exception safe.

this is not really an issue of this task, and I believe we should soon stop
creating the schema in the accessor context in the first place, but the bug
is real so I fixed it anyway.  Assuming this creation code and the older
version of implementation will be gone soon, I simply duplicated the mostly
same code in both old and new implementations.

Also, it seemed extremely difficult (I'd say it's effectively impossible) to
test these cases.  I couldn't come up with any reasonable way to test them,
so I didn't add any test.

[1626] move some more json tests around

[1626] use one list of 'whitespace' chars

and even more tests

Moved a couple of 'basic' tests out of 'escape' and to 'to_and_from_string' test

[1626] Add more testcases

[master] update changelog for merge of #1535

[master] Merge branch 'trac1535'

[1535] use OutOfZone instead of InvalidParameter in findNSEC3()

[1626] check string termination and whitespace fix

whitespace 'outside' of elements is skipped, now including \r and \b

[1626] Add some more testcases

[master] trivial editorial cleanup: removed spaces in a blank line.

[1579] editorial cleanups: shortened long lines, indentation consistency.

[1579] folded a long line

[1579] more suggested editorial fixes: use boolean as boolean

[1579] some more suggested editorial fixes:

- use boolean as boolean: if (is_xx == true) => if (is_xx)
- spacing consistency if (X){ => if (X) { // (many of this in the branch)
- folded a long line

[1579] editorial cleanup, mostly per the guidelines.

- folded long lines
- removed white space at EOL
- spacing consistency.  in particular, add one space after comma
- alignment

[1579] cancel unnecessary change (this will help reduce conflict on merge)

[1535] (editorial) combined two shorter lines into one.

[1535] renamed OutOfZoneFind to OutOfZone

And remove the memory-datasrc specific one

[1535] address review comments

- removed superfluous toText() calls in out-of-zone exception creation
- moved in-memory out-of-zone check to be part of findNode instead of separate check in find()
- added out-of-zone check to addAdditional lookup (and don't lookup unless in zone)
- removed FIND_DEFAULT parameters in unit tests (they are default values)
- change notify_out sqlite test data to have out-of-zone-but-known NS target

[1579] If nsec and nsec3 coexist in zone, find function and findAll
function will throw error. find and findAll function will set
RESULT_NSEC3_SIGNED if the zone is signed with NSEC3.

[master] Move cppcheck suppressions to the source files (inline)

This fixes a cppcheck regression introduced in commit
48cfa00bb988cfeebab4e6ac7ae440cd685137e5 (I think), which caused line
numbers to change.

[1626] fix non-printable escaping in json conv

[1627] Add comment about use of try-catch blocks

[1627] Add a couple more testcases

 * Case when no file is specified
 * Case where both origin and file are not specified

[master] Fix testcase name

[1627] Fix type of exception thrown in testcase

NameParserExceptions are now caught and AuthConfigError is thrown.

[1627] Rewrite code to use local variables

[1627] (editorial) folded long lines

[1627] (editorial) folded long lines

[1627] Add the namestring that triggers the exception to the message

This is so that users see what name caused problems in the log.

[1627] Unify testcases for NameParserException checks

[1627] Check for empty origin and file in config, and throw with more relevant error msg

[1627] fixed some style issues: untabify, position of '*', fold long lines.

bug #1627: Update error message

bug #1627: Generate better error messages when origin is not specified

bug #1627: Rewrite code to use NameParserException

bug #1627: Add common base class for name parser exceptions

[324] missing EXTRA_DIST

[324] make sure dbutil is run only when the DB file exists.

otherwise make install incorrectly fails for a clean install.

[324] cleanup: removed tests checking broken DB missing "diffs" table.

now this scenario shouldn't happen (unless the operator intentionally creates
such DB by hand) because we increased the major version.  These tests
require manually crafted DB files, and I don't think it worth the overhead.

[324] cleanup: sorted messages. (no content change)

[324] use the dbutil script for pre-compatibility checker.
sqlite3-difftbl-check.py is now  unnecessary, so has been removed.

[324] added tests for different schema versions for sqlite3_ds.

this is mostly retired, but we are still using it for the current loadzone
script.  so I re-enabled the test, removed old test stuff, and added this
new test case.

[324] added test cases for different SQLite3 schema versions.

also made a minor cleanup: use char* instead of string in the namespace level.
it should be more robust against initialization fiasco.

[324] upgraded all test SQLite3 data to schema version 2.0

(except for some "intentionally broken" ones).  these were semi-automatically
convered by b10-dbutil.  "nodiffs" needed to be tweaked by hand after upgrade
for their test.  and all unit/system/lettuce tests now passed.

[324] fixed typo in schema def, and make sure setting the minor version.

[324] fixed typo in the new schema definition, and make sure set minor version.

[324] re-increased the assumed latest version dbutil (to 2.0).

test cases that rely on the latest version and were temporarily disabled
were now re-enabled.

[324] make sure latest SQLite3 version (2.0) will be used on creation.

also, in sqlite3 accessor and the old data source API, check both the
major and minor versions, log any mismatches, and throw if major doesn't
match.  right now there's no test for it, which is bad, but this seems to
be best way forward from the given patch.

[324new2] Merge branch 'trac324new' into trac324new2

[1535] (editorial) folded long lines

[master] Add .gitignore files

[1626] remove wrong comment line

[1626] do not escape forward slash in toJSON

[master]Revert "[master] Show testnames when running Python unit tests"

This reverts commit 6a9dbeff7d28a8ef91ceddff5dbcdef9a7d1452e.

Most builder systems use Python 3.1.

This verbosity feature was added in Python 3.2.

[1626] escape JSON strings correctly

[1535] improve OutOfZoneFind text and docstring

[master] Show testnames when running Python unit tests

bug #1819: Remove duplicated code

[1535] make ZoneFinder::find throw on out-of-zone query

Added new OutOfZoneFind exception, which is thrown (instead of NXDOMAIN result) when zonefinder::find() is called for an out-of-zone name (normally this should not happen; the zonefinder object is itself the result of a call to findZone()

[master] Merge branch 'trac1772'

[master] disable SQLite3Update.rollbackFailure test

see ticket #1845

[master]Merge branch 'master' of ssh://git.bind10.isc.org/var/bind10/git/bind10

[master] add Mukund to authors.bind.

[master] changelog maintenance

add date for 20120329 release

use a tab before the keyword type.

use two tabs before the committer username.

[master] regenerate xml from mes files

[master] regen manpages

[master] add a git commit reference

But not specific for the entire merge

[trac1732] add changelog entry

[1535] do find_zone for each NS address in notify lib

[trac1732] datasource type is required

This was copied from manpage so fix there.

Noticed by jelte. And I confirmed:

2012-03-28 11:07:43.931 ERROR [b10-auth.auth] AUTH_CONFIG_UPDATE_FAIL update of configuration failed: Missing data source type

[trac1732] more doc work for loading zones

Add more examples and explanations about in-memory.

Be clear again that b10-loadzone is sqlite3.

[trac1732] remove redundant note about IN class

[trac1732] minor docbook tags formatting changes

[trac1732] add a single section id

TODO: add them for all sections in table of contents for linking

[trac1732] more in-memory datasource docs

Some cleanup, formatting tags added.

Change example from single configuration step (in JSON) to multiple
configuration commands.

More examples.

[trac1732] add auth configurations and commands

This was pulled in verbatim from the b10-auth.xml manual page.
TODO: automate this or rewrite this.

Also this lacks formatting.

[trac1732] start of the documenting in-memory data source use

from jinmei's guide.diff from ticket #1732

[master] added a note about dbutil that there's no use of it yet.

committing based on jabber discussion.

[master] Merge branch 'trac1846'

[master] changelog for #1836

[master] Merge branch 'trac1836'

[master] add some pyo (python optimized files) to CLEANFILES

This is to fix the distclean target to fix the reported
distcheck failures.

These pyo files were generated by running the dbutil for tests.

[1846] temporarily consider v1.0 as current schema version

and disable a number of relevant tests

[1491] added missing test for spec_part_is_named_set