Added doxygen comments in the most important places.

Turn BoostPinger into true async operation

No rescheduling in BoostPinger

Implemented the feature:
- Make the link down interval also configurable. Today it is possible just to
  configure the amount of time the link must to be up to notify. Do another
  configuration to allow the time of link down.

Two TODO entrys solved

Reuse BoostPinger object

Fix ping of IP addresses (don't do DNS lookup)

Solved the item:
- Perform a better treatment of missing configuration items, actually the
  application just crashes.

All items that are strictly required have default values, or are checked against
valid values. The last item missing was nameserver, which has a default value now.

Alternative approach to solve:
- If one of the "[host]" in the configuration file omit the "interval" entry,
  the application uses the next "[host]"'s "interval" entry, instead of a
  default value. The problem is specific to the configuration read classes
  that are not smart enough to verify which "[host]" have this entry and which
  don't.
  Example:
     [host]
     name=A
     [host]
     name=B
     interval=1
     [host]
     name=C
     interval=3
  Thus, the tuples will be (A,1) , (B,3), (C,?), but A should have a default
  value, instead of borrow B's value.

The approach consists of ensure the program only runs if there is an interval
for every single host.

Allowing unknown options and changed the default host down limit to 0

Clean thread shutdown

Remove boost signal handler: It's overdesigned for our use case and creates another thread just to wait for signals

Handle signals in a thread safe way

Convert signal handler library to unix line endings

Create unique identifier for every ping

Moved the boost signal handler library

Updated TODO

The multi-threading approach solves the blocking

Merge branch 'pingcheck' of ssh://aqua.m.i2n/data/git/intranator/source into pingcheck

TODO changes

working signal handler

Renamed project from libpingcheck to pingcheck

Update TODO

Modularization of the main.cpp
- semantic order enforced by parameters
- less spaghetti code
- code more readable
- can be moved to a class more easily

Defining the LinkStatusAnalyzerItem
- to keep the standard
- to provide the same benefits as the other Item types

Defining the type PingSchedulerList as a vector of PingSchedulerItem
- to keep the standard
- same benefits as HostList

Defining the type HostList as a vector of HostItem
- provides a central point to change the container type
- makes the code more readable
- reduces the amount of code to read

Starts pinging immediately, instead of schedule to start "interval" seconds later

Renamed handle_timeout_echo_reply() to handle_timeout_icmp_packet()
- the name is more appropriate once the timeout is not triggered if any kind of ICMP packet arrive, not just echo reply

Using lock_guard to handle mutex locking/unlocking through RAII

The BoostPinger is more picky about received packets, because when running in
parallel with other BoostPingers, it could receive echo replies from any host,
leading to things like:
    26 bytes from 209.85.149.104: icmp_seq=1 ttl=54 time=24 ms   <-- pinger A received packet from host A
    26 bytes from 209.85.149.104: icmp_seq=1 ttl=54 time=1999 ms <-- pinger B received packet from host A??
     - Host up: www.postbank.de                                  <-- what? You didn't receive any packet from the host you had ping
     - Stick to the original ping interval: 10s
     - Time elapsed since last ping: 12s                         <-- OK, pinger A parsed the packet from host A
     - Stick to the original ping interval: 5s
     - Time elapsed since last ping: 6s                          <-- WRONG, pinger B should not parse the packet from host A

Running the BoostPingers in parallel (multi-thread approach instead of assynchronous)

Removed fixed bugs

Added a mutex to avoid data race conditions in the shared link status object

Better comments

Introduce connd pingcheck subsystem based upon NTP subsystem

fooo

Separated the endpoint creation in a method

Build rpm package in parallel

Make the status notifcation cmd optional. Also make the status parameter optional

Switch log level to ::Info during development

Don't bind the source interface if not specified

Comment out -Werror as rpm will compile with -O2 - this results in warnings

Iclude Intra2net commercial license for now

Included doxygen comments to member variables.

Log messages via I2n::Logger instead of cout/cerr.

Split Readme file.
- code conventions moved to Readme.code

PingInterval class changed from template to regular class to make it simple.

Fixed some issues issued by PC-lint

Improvement of ConfigurationReader
- renamed is_generic_options() to halt_on_generic_options(), which describes more precisely the semantic of this method
- moved halt_on_generic_options() to outside the parse_configuration_options()
- config-file processing moved from parse_configuration_options() to parse_generic_options(), because it is part of generic options
- the test "if ( vm.count(xxx) )" is no longer implicit compared to integer

Implemented feature: Program runs as daemon

The speed_up method could set the Interval to zero, which is a bug.
- also added asserts to harden the code and ensure the Interval will never become an invalid value.

Fixed bug: If one ping fails, the subsequent analysis report that the ping is down.
- The reason was that in HostStatusAnalyzer, analyze_ping_failed_count() was
  called from within increase_ping_failed_count(), thus only when the ping
  failed the ExceededPingFailedLimit was updated, to false. There were no path
  where ExceededPingFailedLimit was updated to true.
  Now, the analyze_ping_failed_count() is called everytime we call
  update_ping_statistics(), which updates ExceededPingFailedLimit everytime.

Split the code of DnsResolver::resolve into more methods

Implemented feature: cache DNS requests info only until DNS reply is valid (TTL).
- created a class TimeToLive which is self updatable, and can get the ttl updated
- before each ping checks if the address is valid, if not, builds another IP list (which renews the ttl)

Fixed the type punning from boost::net::dns::message class

Removed one level of indirection provided by PingScheduler::handle_next_ping

Reading the nameserver form configuration file

Non-virtual destructors for classes that are unlikely to be base classes

Using the boost-net-dns library to resolve DNS host names

Commented the -O2 optmization level because it activates the -Wstrict-aliasing warning, which is issued by the boost-net-dns libraries.

Replaced post-fixed by pre-fixed increment

Increased the warning level
- included -pedantic, -Wshadow, -Wcast-qual and -Wconversion warnings
- fixed some warnings regarding lost of precision from conversions
- asserted that intermediate products from arithmetic operations fall within certain range

Using the std namespace within the souce file

Only the "using namespace std" is allowed, classes from other namespaces are included via "using"
- avoid conflicts with the boost DNS library classes

Renamed packages to more appropriate names
- ping to host, because it contains classes to deal with host
- notify to link, contains classes that deal with many hosts, on link basis

Encapsulated the encode, decode, read and write methods within the IcmpMessagePayload class
- simplify the message classes

Fixed more problems issued by PC-lint
- alternative initialization to avoid calling virtual methods from constructor

Fixed more problems issued by PC-lint (3 times now, bad programmer!)

Increment the replies count when receiving a destination unreachable, avoiding the request timeout message

Separate method to print the timeout of a echo request

Found an issue with sequence number

Pinger now handles ICMP Destination Unreachable messages
- handle_receive_echo_reply rename to handle_receive_icmp_packet, once it handles not just echo replies
- timer member variable has a more meaningful name

Refactoring the ICMP message parsing
- IcmpType enum separated from the IcmpHeader class, thus it has its own header without requiring to add the IcmpHeader's header file
- the IcmpHeader class delegates the parsing polymorphically to specialized IcmpMessage classes  (i.e. allowing the parsing of Destination unreachable messages)
- IcmpHeader class does not requires the Payload member variable anymore, once it delegates the parsing to IcmpMessage classes

More meaningful name for timer objects.

Given more meaningful names to the variables

More wish list items

Implemented the requirement "Support selection of the source network interface, so we can change the source ip address when ping"
- unfortunately, I couldn't find a way to bind to an interface through Bosst Asio. So it was required to rely on native Unix Socket interface.

Reading the network interface from the configuration file.

Fixed more problems issued by PC-lint.

Message aesthetical improvement

Removed the address resolution from the pinger.
- There is a special  class to translate DNS to IP, thus makes no sense to keep this feature here, which simplify this class.

Fixed problems issued by PC-lint.

Included the unit in the variable names
- states clear that we expect seconds, minutes or whatever

Implemented the requirement "If we notified an external system that a link is down, also notify it if the link is back up (=stable for a configurable period of time)"
- the status of the link is reported only once, when it changes from down to up, or vice versa
- allows to configure the minimum amount of time the link must stays stable in order to report its status

Found another BUG (its capitalization gives a hint about the severity of this one)

Standardizing the enumerations

Renamed stable-link-up-interval to link-up-interval

Fixed return value, should be string, not boolean

Included boost system to the required libraries

Renamed StatusNotifier to LinkStatusAnalyzer
- the main purpose of this class is to decide whether the link is down or up, by keeping track the amount of hosts down.
- renamed the variable Analyzer to HostAnalyzer to avoid confusion with the LinkAnalyzer

Renamed PingAnalyzer to HostStatusAnalyzer
- this describes better what the class is suppode to do, analyze the status of the host, using the pings.

Add TODO entry

Read the configured interval that a link must be up to notify

Placed limit qualifier in the end of variables names (i.e. xxx_limit instead of limit_xxx)
- this keeps a pleasing symmetry with other qualifiers, for example xxx_count, xxx_limit, xxx_index

Renamed PingAnalyzer's update_ping_failed_statistics() to analyze_ping_failed_count()

Improved asserts

Implemented the requirement "Intelligently retry if a host doesn't reply: Try it more often"
- PingAnalyzer::exceeded_ping_failed_count_limit() made public, thus the Analyzer can check when is required to speed up the ping (or back to the original interval)
- Included a PingInterval template class to hide the computation logic of interval speed up and slow down

Attributes names according to the company's coding style.

Renamed PingStatusNotifier to StatusNotifier
- the class scope is not restricted to ping status notification, it is
  responsible to notify the system when certain amount of hosts are down,
  independently of the number of pings given to that host.

Included script to notify when system is down or up