[trac4480] Update per review comments

Create a LogContentTest class and move the functions to manipulate
a test to check on the log output there.  This will make it available
for other tests in the future.  As part of this bundle a bit more
work into the class to minimze the calls in the test routines.

Where reasonable remove extra copys of the values being pushed
onto the stack - instead of generating a copy just for the logging
simply use the value from the top of the stack directly.

Swtich to using single quotes around text values.

Added some more description in the classify document

[4480] Add debug logging for evaluation of classification expressions

This patch adds a set of log messages to indicate what values are
being popped from or pushed to the classification value stack.  This
is meant to be used by an administrator when debugging the expression
statements.

[master] Added ChangeLog entry for #4485

[master]Merge branch 'trac4485'

[master] Corrections in the config examples.

[master] ChangeLog updated after 4112 merge.

[master] Merge branch 'trac4112' (subnet4o6 selection based on v6 info)

[4112_rebase] Compilation fix.

[4485] Change type of variables to overcome compiler objections

One compiler was objecting to placing the address of a size_t variable
into a variable declared as "unsigned long*".  Changing the
declaration from "size_t" to "unsigned long" fixes the problem.

[4112_rebase] Minor change after review

 - check for address being :: updated

[master] Added ChangeLog entry 1112 for #4492.

[master] Fixed unit test compilation post merge

    HooksManager::loadLibraries() call now requires
    LibraryCollection as parameter

[master]  Server destructors now explicitly unload hooks libraries

    Merge branch 'trac4492'

[4492] Addressed review comments

src/bin/dhcp4/tests/hooks_unittest.cc
src/bin/dhcp6/tests/hooks_unittest.cc
    Fixed includes, copyright, and removed unnecessary calls to
    IfaceMgr::instance().deleteAllExternalSockets();

[4492] Addressed review comments

src/bin/dhcp4/tests/hooks_unittest.cc
    TEST_F(LoadUnloadDhcpv4SrvTest, unloadLibaries)  - new test that
    verifies Hooks libraries are unloaded by the Dhcpv4Srv destructor

src/bin/dhcp6/tests/hooks_unittest.cc
    TEST_F(LoadUnloadDhcpv6SrvTest, unloadLibaries)  - new test that
    verifies Hooks libraries are unloaded by the Dhcpv4Srv destructor

[4112] 2 new unit-tests added.

[master] Minor correction in doxygen comments.

 Thanks to wpk for reporting this issue.

[4112] selectSubnet4o6 implemented + one unit-test

[master] Added ChangeLog entry for #4493.

[master] Merge branch 'trac4493'

[4493] Optimize lookup of timed out packets.

[4493] Updated statistics calculation.

Presented rate is now the number of ACK/Reply messages,
rather than DO/SA.

[master] ChangeLog updated after #4297 merge.

[master] Merge branch 'trac4297' (hook libs can now take parameters)

[4493] Extended perfdhcp test for collected packets.

The extended test verifies that the stats manager is
still usable after some packets have been collected.

[4493] Improve packet collecting mechanisms in perfdhcp.

When the perfdhcp performs unordered packet lookup within
a bucket of sent packets, it will remove all timed out
transactions.

[master] Corrected invalid parameter passed to the test function.

This is to eliminate some of the recently found covertity isssues.
Reviewed on jabber.

[4492] Explicitly unload hook libraries prior during shutdown

A call to HooksManager::getHooksManager().unloadLibraries() was added to
the destructors Dhcpv4Srv::~Dhcpv4Srv() and Dhcpv6Srv::~Dhcpv6Srv(). This
ensures hook libraries are unloaded prior to static objects.

[4297] spelling

[4303] Added Changelog entry for #4303.

[4297] Corrected headers installation dir.

[4297] Addressed review comments:
 - returnining different values in callout_params_library.cc
 - returned values are now described for getParameter
 - corrected index syntax comment in getParameter
 - added missing include
 - hooks_user.dox updated
 - hooks unit test moved, so it is now next to other hooks tests

[master] Merge branch 'trac4303'

[4303] Removed unnecessary variable initialization.

[4303] Address second round of review comments.

- No additional check for HR mode in findReservations
- Additional commentary in the loop collecting host ids.

[master] Added ChangeLog entry 1108 for #4239

[master] Unit tests now use production database scripts

    Merges in trac4239.

[master] Updated ChangeLog

[master] regen

[4303] Added commentary in dhcp4_srv.h

[master] Finish merge of trac4268a (DHCPv4 fields)

[4303] Addressed remaining review comments.

[4303] Renamed CfgHostReservations to CfgHostOperations.

This change was suggested during a ticket review.

[trac4268] Tidy up some documentation strings

[4268a] Fixes and cleanups

[4268a] Updated tests for 4 byte padding

[4268a] untabify

[4268a] Padded hlen/htype pkt4

[4239] Addressed review comments

src/lib/dhcpsrv/pgsql_lease_mgr.cc
    Amended commentary for getColumnValue() variants
    Added decription of Uiaid union
    Changed std::system to ::system

src/lib/dhcpsrv/testutils/mysql_schema.cc
    Changed std::system to ::system

src/lib/dhcpsrv/testutils/mysql_schema.h
    Fixed typo

src/lib/dhcpsrv/testutils/pgsql_schema.cc
    Removed extraneous include
    Changed std::system to ::system

[4268a] Added extracted field lengthes

[master] Update changelog with commit number for 4269

Merge branch 'trac4269' Merge 4269 after updates to fix conflicts

[trac4269] Updates per review comments, basically change some variable names

[4303] Values in 'host-reservation-identifiers' used by the servers.

Both DHCPv4 and DHCPv6.

[4303] Implemented parser for 'host-reservation-identifiers' list.

[4303] Added class holding configuration of host identifiers.

The identifiers are held on an ordered list and the server will
use them to search for host reservations for each client using
this order.

[4303] Servers now search for host reservations using any identifier.

[4239] Moved schema scripts to src/share/database/scripts

    Moved schema scripts into a new common location:

        src/share/database/scripts

    to make them accessible to both kea-admin and dhcpsrv testing

[master] Added ChangeLog entry for pull request #19.

[master] Merge branch 'github19'

[github19] Updated AUTHORS file.

Added Angelo as contributor.

[4239] Added drop scripts for both MySQL and Postgresql

Rather than use hard-coded lists or query logic for dropping the
database all MySQL and Posgresql tests use new drop scripts
added to src/bind/admin/scripts.

src/bin/admin/scripts/mysql/dhcpdb_drop.mysql
    - New SQL script to drop a MySQL database

src/bin/admin/scripts/pgsql/dhcpdb_drop.pgsql
    - New SQL script to drop a Postgresql database

src/bin/admin/tests/mysql_tests.sh.in
    - mysql_wipe() - modified to use new drop script

src/bin/admin/tests/pgsql_tests.sh.in
    - pgsql_wipe() modified to use new drop script

src/lib/dhcpsrv/tests/mysql_lease_mgr_unittest.cc
    - TEST(MySqlOpenTest, OpenDatabase) - added show_err=true
    to destroyMySQLSchema() calls

src/lib/dhcpsrv/tests/pgsql_lease_mgr_unittest.cc
    - TEST(PgSqlOpenTest, OpenDatabase) - added show_err=true
    to destroyMySQLSchema() calls

src/lib/dhcpsrv/testutils/mysql_schema.cc
    - destroyMySQLSchema(bool show_err) - modified to use the new
    drop script and accept show_err parameter

src/lib/dhcpsrv/testutils/pgsql_schema.cc
    - destroyPgSQLSchema(bool show_err) - modified to use the new
    drop script and accept show_err parameter

[4239] MySQL and Postgresql unit tests use production schema create scripts

src/lib/dhcpsrv/testutils/schema.h
src/lib/dhcpsrv/testutils/schema.cc
    New files that contain constants and functions common to RDBMS
    testing

src/lib/dhcpsrv/testutils/pgsql_schema.h
src/lib/dhcpsrv/testutils/pgsql_schema.cc
    New files that contain constants and functions needed for
    Posgresql testing

src/lib/dhcpsrv/testutils/mysql_schema.cc
src/lib/dhcpsrv/testutils/mysql_schema.h
    Removed common constants and functions
    Added MYSQL_VALID_TYPE
    runMySQLScript() - new function to submit a script to MySQL

src/lib/dhcpsrv/pgsql_lease_mgr.cc
    PgSqlLease6Exchange - replaced uint32_t iaid member with Uiaid union
    instance.  This permits the value to be safely stored in the database
    as an INT (signed 4-byte value)
    getColumnValue()- added int32_t variant of this method

src/lib/dhcpsrv/tests/Makefile.am
    Removed schema_pgsql_copy.h

src/lib/dhcpsrv/tests/mysql_host_data_source_unittest.cc
src/lib/dhcpsrv/tests/mysql_lease_mgr_unittest.cc
    Altered tests to use MYSQL_VALID_TYPE

src/lib/dhcpsrv/tests/pgsql_lease_mgr_unittest.cc
    Moved constants and functions testutils/pgsl_schema.h & cc

[github19] Integrated pull request 19 with unit test fix.

[4239] Fix constraint-based unit test failure

src/lib/dhcpsrv/tests/generic_lease_mgr_unittest.cc
   - GenericLeaseMgrTest::testLease6HWTypeAndSource() - now uses valid values
     for hardware address source

[4239] MySQL unit tests in dhcpsrv now use production db create script

src/lib/dhcpsrv/tests/mysql_lease_mgr_unittest.cc
    TEST(MySqlOpenTest, OpenDatabase) - pass in show_err=true for call
    to createMySQLSchema()

src/lib/dhcpsrv/testutils/Makefile.am
    - Added definition of TEST_ADMIN_SCRIPTS_DIR
    - removed schema_mysql_copy.h

src/lib/dhcpsrv/testutils/mysql_schema.cc
    - Removed include of schema_mysql_copy.h
    - destroyMySQLSchema() - now contains destroy_statment[] formerly
    defined in schema_mysql_copy.h
    - createMySQLSchema() - Creates the database through command line
    invocation of MySQL cli, mysql, passing it the production database
    creation script

[trac4269] Update to put 4 byte strings on the value stack for integers

[master] Update change log to add commit number for trac4265

[master] Added ChangeLog entry 1104 for #4259.

[master] DDNS parameter, replace-client-name, now supports multiple modes

    Merges in trac4259.

[4259] Address review comments, part II

    Fixed minor typos.

[master] Added ChangeLog entry for #4301.

[master] Merge branch 'trac4301'

Merge branch 'trac4265' Access to options and constants from DHCP6 relay encapsulations

[trac4265] Update copyrights and use HOP_COUNT_LIMIT instead of hardcoded value

[4301] Addressed review comments.

[4259] Addressed review comments

    Moved string/enum conversion functions inside D2ClientConfig

    Changed underscores to hypens in replace-client-name values and changed
    all literal values to lower case.

    Fixed minor wording issues in admin guide and new log messages.

    Added commentary and extra error logging to unit tests

    Added test case for invalid value of replace-client-name

[master] ChangeLog for trac4293

[master] Merge branch 'trac4293'

[4301] DUID and HWAddr classes use strutil to convert text to binary.

[4301] Implemented utility functions to convert from hex strings.

The supported hex strings may have different formats:
- colon separated values,
- with '0x'prefix
- no prefix, no colons

[trac4265] update unittests to go along with token name change

[trac4265] Regenerate parser.cc and parser.h after updates

[trac4265] updates per review comments

[4265] Removed whitespaces

[4293] Correction of a typo in the dhcpsrv message file

[4301] HostReservationParser::isIdentifierParameter belongs to parent class.

[4301] Host configuration parser now supports circuit-id parameter.

[master] Correcting previous change.

Replace <cstdint> with <stdint.h> because the former is C++11 only.

[master] Added missing inclusion of <cstdint> in strutil.h.

Okayed on jabber.

[master] Added ChangeLog entry 1100 for #4243.

[master] Mitigate GCC 5.x optmizer issue in asiolink

    Merged in trac4243.

[master] Merge branch 'trac4300'

[4300] Addressed review comments.

[4243] Distcheck failure correction

    Distcheck sporadically fails to clean up test_leases.csv.2
    can be left behind by dhcp4 and dhcp6 testing.

src/bin/dhcp4/tests/Makefile.am
src/bin/dhcp6/tests/Makefile.am
    - Added test_leases.csv.* to CLEANFILES

[4243] Addressed review comments

src/lib/asiolink/Makefile.am
    added asio_wrapper.h to list of sources

src/lib/asiolink/asio_wrapper.h
    updated commentary to include GNU/Boost bugs and
    added prominent warning comment

[trac4265] Add test code, tidy up main code

[4259] Updated admin guide and spec files

    Revamped discussion of FQDN name generation and use of the
    replace-client-name parameter in the v4 and v6 guides.

[4259] kea-dhcp6 now supports replace-client-name modes

    src/bin/dhcp6/dhcp6_messages.mes
        - Added new log message, DHCP6_DDNS_SUPPLY_FQDN

    src/bin/dhcp6/dhcp6_srv.cc
        - Dhcpv6Srv::processClientFqdn() - modified to support the name
        replacement modes

    src/bin/dhcp6/tests/fqdn_unittest.cc
        - FqdnDhcpv6SrvTest::testReplaceClientNameMode() new method which tests
        a server's handling of a single client packet for a given
        replace-client-name mode.

        - TEST_F(FqdnDhcpv6SrvTest, replaceClientNameModeTest) - new test which
        exercises the permutations of client packets and replace-client-name
        modes.

[4259] kea-dhcp4 now supports replace-client-name modes

    src/bin/dhcp4/dhcp4_messages.mes
        - Added new log message DHCP4_SUPPLY_HOSTNAME

    src/bin/dhcp4/dhcp4_srv.cc
        - Dhcpv4Srv::processClientName() - pushed lack of host option in the
        client request down into processHostoption()

        - Dhcpv4Srv::processHostnameOption() - modified to support the new name
        replacement modes

    src/bin/dhcp4/tests/fqdn_unittest.cc
        - NameDhcpv4SrvTest:: testReplaceClientNameMode() - new method which
        tests a server's handling of a single client packet for a given
        replace-client-name mode.

        - TEST_F(NameDhcpv4SrvTest, replaceClientNameModeTest) - new test which
        exercises the permutations of client packets and replace-client-name
        modes.