[4276] Addressed review comments

Minor cleanups, added some unit testing of PgSqlExchange functions.

[3572] Test assignment of host specific options with DHCPINFORM.

[3572] Test that vendor specific options are overriden.

In the new test the vendor specific options are defined on the
host and global levels. Host specific options should override
globally defined options.

[3572] DHCPv4 server sends host specific options.

[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] ChangeLog for trac3164

[master] Merge branch 'trac3164'

[4510] Fix for escaped characters (e.g. backslashes) in option contents.

[3164] Add changes suggested by review

Change the connection timeout parameter from an "int" to an "unsigned
int". Update the checks to allow for lexical_cast not throwing an
exception when converting a string representing a negative number
to an unsigned int.

[4318] Added unit tests for reservations by DUID in DHCPv4.

[master] Added ChangeLog entry for #4495.

[master] Merge branch 'trac4495'

[master] Added ChangeLog entry for #4499.

[master] Merge branch 'trac4499'

[4308] Subnet selection in v4 tweaked, added two debug messages.

[4308] Implemented subnet selection based on interface name.

[4498] Removed support for callback functions unpacking options.

Previously we used callback functions to pass a pointer to the
server-side function unpacking options. Currently we register
runtime option definitions in libdhcp++, so the callback
functions are not necessary anymore.

[4498] Unified container types holding option definitions in LibDHCP.

[4498] Avoid copying container with option definitions during parsing.

[4276] Replace use of PGresult with PgSqlResult throughout

[4498] Extended test for parsing vendor options.

This test checks that the vendor specific option definitions are
used when parsing vendor option.

[4498] Removed server specific function for parsing DHCPv6 options.

[4276] Created new base class, PgSqlExchange

src/lib/dhcpsrv/Makefile.am
    Added pgsql_exchange.cc and pgsql_exchange.h

src/lib/dhcpsrv/pgsql_exchange.h
src/lib/dhcpsrv/pgsql_exchange.cc
    New files, containng new base class PgSqlExchange from
    which was distilled from PgSqlLeaseExchange

src/lib/dhcpsrv/pgsql_lease_mgr.cc
    Refactored exchange classes to use new base class
    Moved PsqlBindArray into pgsql_exchange.*

[4498] Removed server specific method for unpacking DHCPv4 options.

[4510] Added unit-test that replicates the issue of mishandling backslashes.

[4499] Removed inefficient conversion of the host identifier to text.

[4276] Initial impl of PgSqlConnection class

    Initial refactoring of Postgresql connection logic out of
    PgSqlLeaseMgr into new PgSqlConnection.

[4495] Fixed inefficient creation of DHCP Message Type option.

[4281] Renamed MySqlHostExchangeOpts to MySqlHostWithOptionsExchange.

[4281] Updated recently modified files with proper commentary.

Also, fixed a couple of minor issues.

[4281] Creation of a new host in MySQL is wrapped in transaction.

[master] Added ChangeLog entry 1115 for #4481

[master] Added query<x> arugment to several callouts

    Merge branch 'trac4481'

[4481] Changed lease6_select API doc to show query6 as "in" only

[3164] Updated kea-admin copyright date

[4281] It is allowed to use formatted option values in the host database.

[4481] Add missing description of query6 argument to lease6_select

[3164] Correct a typo in the user guide

[4281] Extended MySQL host data source with DHCPv4 and DHCPv6 options.

Missing things:
- proper commentary
- formatted values for options
- rollback host insertion when failing to add reservation or option

[master] Added ChangeLog entry for #4485

[master]Merge branch 'trac4485'

[3164] The database connection timeout is now a configurable parameter

In addition, the default has been changed to five seconds.

[4481] Made query4 and query6 callout arguments uniformly available

The client packet is now uniformly available to all client packet driven
callouts for both v4 and v6:

Added "query4" to lease4_select and lease4_renew
    src/bin/dhcp4/dhcp4_hooks.dox
        Added query4 argument to lease4_select and lease4_renew documentation

    src/bin/dhcp4/tests/hooks_unittest.cc
        Revamped to track both query4 and response4 arguments
        TEST_F(HooksDhcpv4SrvTest, lease4RenewSimple) - modified to verify
        query4 set by lease4_renew callout

    src/lib/dhcpsrv/alloc_engine.cc
        - AllocEngine::createLease4()
        - AllocEngine::reuseExpiredLease4() - modified to add query4 to
        lease4_select callout arguments
        - AllocEngine::renewLease4() - modified to add query4 to lease4_renew
        callout arguments

    src/lib/dhcpsrv/tests/alloc_engine_hooks_unittest.cc
        HookAllocEngine4Test() - modified to track query4 argument
        TEST_F(HookAllocEngine4Test, lease4_select) - modified to verify query4
        callout argument

Added "query6" to pkt6_send and lease6_select
    src/bin/dhcp6/dhcp6_hooks.dox
        Added query6 argument to pkt6_send and lease6_select documentation

    src/bin/dhcp6/dhcp6_srv.cc
        Dhcpv6Srv::processPacket(Pkt6Ptr& query, Pkt6Ptr& rsp) - modified to
        add query6 to the pkt_send callout arguments

    src/bin/dhcp6/tests/hooks_unittest.cc
        Revamped to track both query4 and response4 arguments
        TEST_F(HooksDhcpv6SrvTest, simplePkt6Send) - modified to
        verify query6 set by pkt6_send callout

    src/lib/dhcpsrv/alloc_engine.cc
        - AllocEngine::createLease4()
        - AllocEngine::reuseExpiredLease6() - modified to add query6 to
        lease6_select callout arguments
        callout arguments

    src/lib/dhcpsrv/tests/alloc_engine_hooks_unittest.cc
        HookAllocEngine6Test() - modified to track query6 argument
        TEST_F(HookAllocEngine6Test, lease4_select) - modified to verify
        query6 argument

[4106_rebase] Compilation fix after rebase.

[4106] Fixed DHCP4o6 for compiling, Linux and OpenBSD

[4106] Changed to use the private options

[4106] Ported DHCPv4-over-DHCPv6 IPC code from fd4o6 private branch

[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

[4281] Alterations to MySQL host database schema.

- Option code in dhcp6_options table is now SMALLINT.
- Subnet ids are now unsigned.

[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.