GitLab

- use the initialized zone name for the owner instead of the hardcoded one
- removed a redundant return
for now these don't affect the test results, but it's cleaner and may
have a substantial difference in future extensions.

- log client's address (needed to update _guess_remote also)
- log xfr type (currently it's always 'AXFR', but we'll soon support IXFR)
- log the event of quote failure

XfroutSession().  The original code was already buggy in this sense, but
with the newer data source API it will be a bit more likely to happen
due to the generality of the API, so it would make sense to fix it here.

one question.

can happen from a broken data source)

instead of the old API.  Now we can completely drop
_create_rrset_from_db_record, so did it.  With this change the main xfrout
code is free from the old API.

not in _reply_xfrout_query() using the old API.
_create_rrset_from_db_record() is now almost obsolete, so the test for it
was also removed.

use that for logging.  get_query_zone_name/class() are now removed
(they were almost one-liner, and didn't see make much sense to be separate
methods other than making the code rather unredable)

using the new API.  _zone_has_soa() and _zone_exit() were not needed anymore,
so were removed, so were corresponding tests.

in _check_xfrout_available().

localstatedir.  (e.g.) /usr/local/var/bind10-devel instead of /usr/local/var.

also updated the EOL code of auth/spec_config.h.pre.in from CR+LF (DOS style)
to LF (unix style), mainly for consistency.

simply use the spec default values in initialization of UnixSockServer.
this way we can avoid redundant initialization.

and log messages for them.  this is necessary to gracefully die due to
broken config data in b10-config.db at startup time.

transfer_acl (for consistency) and is now used as the default transfer ACL.
related tests were added.  also removed DEFAULT_RRCLASS, and retrieved the
default RR class from the spec.

ACL will be used when configured.
An unrelated change was piggy-backed: the ACL check was moved outside of
the try-except block.  it doesn't make sense to return FORMERR when an
exception is raised in the ACL check.
Also some minor style fixes were made (folding some long lines)

best matching ACL for the given zone.

into a single function and provide capability of specifying the ACL context.

generic, the intent is to use it for transfer ACL per zone.  The notion of
per-zone configuration should eventually be implemented in a more generic
way (not specific to xfrout).

also added assertEqual checks to catch such naive redundant copies in case
that happens again.

values for socket.getaddrinfo().
commiting and pushing it at my discretion.

... and use it in the Python unit tests that generate logging output.

so that we can merge this branch as discussed in the planning call today