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

Added the requirement "If XX servers don't reply, it has to notify another system (e.g. execute a script)"
- moved the class PingStatusNotifier from ping to notify directory.
- included a new class that encapsulate the process of system notification

Linking against libi2ncommon

Renamed PingStatusNotifier::alert_host_up/down() methods to notify_host_up/down()

Added intelligence to know how many hosts are down

Const-ification

Reorganization of cmake files
- included package build
- layout change (don't know why package system was complaining about white spaces in the commands)
- redistribuition of the commands across the files

Included spec file to build the RPM package

Removed UML file, because it is out-dated and is not longer required.

Removed Pinger abstract class, thus avoiding the "design ahead" issue
- BoostPinger do not inherit form Pinger anymore
- reference from Code Complete 2nd edition, page 146, item "Be suspicious of base classes of which there is only one derived class"

Changed PingStatusNotifier interface, it is now an object shared among N hosts
- this shared property allows to know how many hosts are down, once we can keep track of all hosts
- only one PingStatusNotifier instance for all PingAnalyzers objects, they pass the host to identify themselves within the notifier object

Added status-notifier-cmd options to indicate which command to call when the system is down/up.

The limit-ping-fail option is available (and read) from the configuration file
- this option sets the threshold of pings to a host that must fail in order to mark the host as down.

Renamed the limit-to-notify configuration option to limit-hosts-down, because it is a better description of the intention.

Replaced uint by int.
- documented that a integer is non-negative using assertions, not unsigned int (uint).

Handle the error when a host cannot be resolved
- reorganization of PingScheduler to do not ping a host that was not resolved
- from now, the DnsResolver::resolve() method must be explicitly called by the client of the class

Errors flushed through cerr not cout.

Included an invalid host and a host that does not reply.

More bugs discovered and annotated

Uses the ping analyzer to evaluate the ping statistics after each ping.

Returns the amount of IPs that were resolved for the DNSResolver host.

Returns a boolean indicating if the ping was successful or not.

Bring aboard the ping analyze and notification classes.
- PingAnalyzer implements the requirement to analyze if there are multiple dns entries for a host, only call this host "failed" if can't reach any of these multiple entries.
- PingStatusNotifier implements the requirement to notify an external system that a link is down, also notify it if the link is back up.But needs improvements.

Replaced the times-to-ping parameter to simplify the code, now it pings just one time and uses a status on/off approach to check if the ping was performed.
- also BoostPinger::create_echo_request() method is const and self sufficient, use parameter instead of changing the SequenceNumber member variable

Re-organizing the headers files in order to...
- avoid hidden dependencies, by including the class header file first.
- keep a inclusion order of C headers, C++ headers, Boost headers and Project headers.
- be relative to the source directory, this keeps clear which package the source file belongs.

Explicit constructor to avoid implicit constructor-conversions-calls

PingScheduler now pings hosts IPs cyclically
- included a DNS resolver member variable to keep track of resolved IPs
- PingScheduler's constructor takes specific arguments instead of a generic Host object, this states clear which data the PingScheduler needs

Bring aboard the first scratch of a DNS resolver
- given a host DNS, this class resolves and keeps a list of IPs for that host

Aesthetic change to header files according to the project standard

Code improvements on BoostPinger
- included an assert to make sure a non-empty address is sent to the socket
- better names for the handlers methods

Moved Host class from ping directory to config directory, because it is a configuration item

Added TODO file, including known bugs report

More descriptions of the configuration in the Readme file

The echo reply timeout can be configured through parameter to BoostPinger

Renamed PingCheck class to PingScheduler.
- this name describes better the main purpose behind this class, which is to call the pinger in configured intervals

The application now pings many hosts in parallel
- uses an asynchronous approach, based on boost::asio::io_service::run() loop and asynchronous timers

Attribute renamed according to the code conventions

Included a section explaining the basics of the code

Code improvements
- stratified some functionalities into other methods;
- const-correctness
- added variables to clarify code

Removed unused class: PingManager

Removed unused header files

Fixed assertion range on times_to_ping