Using inheritance to commute between pingers

Modified configuration file

Renamed encode/decode to encode16/decode16
- specify the number of bits it encodes/decodes
- planning support for encode/decode of 32 bits used by TCP header

Replaced the assert by a conditional
- because the amount of data received can differ from the amount we expect, but this cannot make the application to abort.

Bring aboard TCP pinger class

Moved the icmppinger from host to icmp folder

Included a precondition to DnsResolver::handle_ip_address()

Added a tip in the Readme

Renamed BoostPinger to IcmpPinger
- the TCP pinger will be another class to avoid overloading this one with TCP stuff
- the name looks better =)

Minimum version required of Boost is 1.44

First public commit

Don't package files from kdevelop

Started PC-Lint settings file

Improve packaging: Include correct license files

Add GPLv2 + linking exception to the source

Remove t2n call stuff

Add boost license + proper copyright were needed

Update TODO list

Properly handle ICMP packet replies: The kernel sends us a copy of all incoming ICMP messages on every ICMP socket

Turn RepliesCount into ReceivedReply flag

Update TODO file

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.