added limit to DNS resolution attempts before pinging; report host down to hostatus if exceeded; simplified pingscheduler

fixed bug in ICMP Packet Factory: do not continue with garbage packet if read from istream failed!

removed commented lines that do same as shared_ptr.reset which was added in previous commit

stop creating icmp packet parts from istream if earlier part failed

changes in logging output: added information and changed priority

For command line options for logging output level and target, add available options to help string

updated Readme: added option ratio-random-hosts, mentioned range format in ping-interval, corrected description interface behavior

changed initial log level from Info to Notice to avoid printing of complete config into syslog

changed format for interval specification in pingcheck.conf from "a...b" to "a..b"

changed escaping of shell args in status notifier command: escape each arg separately instead of the whole argument string at once

added two more options to pingchecker: use random interval per host in given range, use random subset of hosts

added doxygen text ofr new variable FirstDelay in PingcheckScheduler constructor

converted a few more BOOST_FOREACH to const reference versions in main

added yEd graph with most important calls

Use const reference BOOST_FOREACH version

introduced delays between starts of pingers, so they are evenly distributed in time right away

if DNS request at start of ping fails, re-schedule further requests; originally failed at once and never tried again

add try-catch to dns resolving of name server to get more precise error

fixed a bug in status notification (first status was sent always)

Documentation: explains how to setuid the pingcheck to allow it to be ran by an unprivileged user.

Test: added testcase for log-output configuration option.

Feature: allow to choose among two log destinations, console or syslog.

Fix: standardized the logfunc header inclusion.

Fix: added src/config/option directory to Doxygen build path.

Changes introduced in master branch.

Changed TCP ping method.
- SYN segments pass easily through firewalls and it is replied both when the
  port is open as well as when it is closed, serving better to the purpose of
  checking if the link reaches a given host. Whereas ACK segments do not get
  replied if the other end is closed or has a firewall in the middle.
- A thoroughly discussion regarding SYN x ACK is available at
  http://www.networkuptime.com/nmap/page4-4.shtml
  http://www.networkuptime.com/nmap/page4-5.shtml

Replaced the boost::Scoped_array by std::vector as the underlying message payload byte sequence.
- This change simplifies many operations.

Fix: the boolean logic was inverted.
- When fail() returns false, the read() and write() methods should return true;
- This avoids the annoying "Error: Could not read ICMP packet" and "Error: could not read TCP Segment." messages.

The application now recognizes the ICMPv4, TCP_IPv4, TCPv4 and TCPv6 ping protocols strings.

Pass the PingProtocolList by const reference, not by value.

Changed the network interface rules, the correct behavior is for each host
that has its source-network-interface equal to "default"
(e.g. source-network-interface=default) is supposed to use the global
default-source-network-interface item.

Update example config to current config options

Fix: src\dns\dnsresolver.cpp  272  Warning 616: control flows into case/default

Mask some PC-Lint messages. Simple constification

Changes introduced in the ip-version-drop branch.

Merge branch 'ip-version-drop'

Conflicts:
src/config/host.cpp
src/config/host.h
src/host/pingscheduler.h
src/main.cpp
test/test_configurationcommandline.cpp
test/test_configurationoptions.cpp

Fix: ip::address::is_unspecified() is available only in boost versions above 1.47.0.

Fix: do not assert when the address has more than one CNAME.

Changes introduced in the nic-per-host branch.

Handling more kinds of errors during file parsing.

Fix: consider the network interface from each host.
- It was keeping the same network interface (from the first host) for all hosts;
- Reduced the variable scope so it is assigned on every loop iteraction.

Removing LocalNetworkInterfaceName attribute, it is no longer used in this class.

Use the "host.source-network-interface" only if the global "default-source-network-interface" is empty.

Test: updating the test cases with the new network interface options.

Documentation: updating the Readme with the new network interface options.

Renamed the global "source-network-interface" to "default-source-network-interface".

Added the host.source-network-interface configuration option.

Passing the protocol list by const reference.

Added a method specially to initialize the pinger object for the first time.

Fixed misspelled include guard.

Revert "Configuration: added the ping protocol fallback count."

This reverts commit f1284dcbaa750c1ffb517ed8e2e6a0a051a36057.

Revert "Test: updating test cases to use the ping protocol fallback feature."

This reverts commit 8e693035e02c8f6cb780cf594e41862fcd71ce2b.

Revert "Perform N pings using one protocol, then after those N pings, switch to"

This reverts commit 46531881f06af5666d62a74836b1101cdcb779d9.

Revert "Documentation: updating Readme and TODO files."

This reverts commit d3c172383ff9ccaff5dc510fea439284f9f28650.

Fix: cast to integer because some compilers do not provide automatic conversion.

Removing the const qualifier to link with Boost.
- Older versions of GCC does not link with different constness.

Not implemented according to the message:

From: Thomas Jarosch <thomas.jarosch@intra2net.com>
To: guilherme.maciel.ferreira@gmail.com
Subject: Re: TODO list
Date: Wed, 7 Mar 2012 11:26:19 +0100

Guten Morgen Guilherme,

On Wednesday, 7. March 2012 11:14:28 Guilherme Maciel Ferreira wrote:
Dude, this seriously sucks. I have some good news for you:
Just completly skip the QoS stuff, we don't need it right now.
Hope your laptop wasn't in the car...

Cheers,
Thomas

Documentation: updating Readme and TODO files.

Perform N pings using one protocol, then after those N pings, switch to
the second protocol, if it available.

Test: updating test cases to use the ping protocol fallback feature.

Configuration: added the ping protocol fallback count.
- This item tells how many times it must ping using one protocol before use the other.

Using boost::asio::ip::address class instead of std::string in the HostAddress.
- That class provides more functionality, so keep it until someone requires a string.

Added the protocols supported in the Readme file.

Undoing commit e58d750735048b76721ade950d3b9e0c75d201f7.
- Using the io_service reference instead of the shared_ptr, because it caused a
double freed error, because it was a stack object in the PingScheduler
(boost::asio::io_service IoService) and a smart pointer elsewhere,
thus, the smart pointers were releasing the stack memory. And once the
PingScheduler didn't had a counter by itself, the program was dumping
in the PingScheduler destructor.
  Once the Boost.Asio library uses the reference itself (e.g.
basic_raw_socket(boost::asio::io_service& io_service, )), I decided to
keep like it was before.

The DNS address resolution is synchronized with the protocol change.
- In this way, when the new protocol requires a different IP version, the DNS
resolver is called again to get the according Address Resource Record.
- Still needs some intelligence.

Repositioning of the ping_done_handler() method according to the order in the header.

Log message reformatting.

Bring aboard a factory to create DNS resolvers.

Added AAAA resource records in the DNS query.
- DNS resolve to IPv6 addresses.

Moved DnsResolver from PingScheduler to PingRotate.
- Simplify the PingScheduler class;
- Makes easier for the PingRotate to match the correct Pinger to the IP version.

Code improvement:
- Replaced io_service references by shared pointers, because
  this ensures that this object is shared.

Code improvement:
- using STL rotate algorithm;
- moved part of ping_done_handler() method to another method to improve readability.

Code improvement:
- using STL rotate algorithm;
- and parameterized the fill_answers_list() method.

Hardening the match methods through asserts.

Using the list of protocols in the scheduler.

Factory method to create the Pinger object that handles a list of protocols.

The configuration supports more than one protocol per host.
- The protocols must be separated by comma, like ping-protocol=ICMP,ICMPv6.

Added a new class to exchange protocols.
- It simply hides the Pingers objects from the scheduler, changing the protocols at
  will. This process is transparent to the PingScheduler.

Documentation: documenting the parsing function.

Added PingProtocolList type.

Replaced boost::shared_ptr<Pinger> by PingerItem.

Small formatting.

Created a HostAddressList type as an alias to the STL list of HostAddresses.

Fix: cast to integer because some compilers do not provide automatic conversion.

Refactor: moved the 'A' resource record filling to a dedicated method.

Removing the const qualifier to link with Boost.
- Older versions of GCC does not link with different constness.

Created a PingItem type as an alias to the smart pointer.

Changes introduced in the test branch.

Implemented in commit aafde4f869c9a119bfb44e54961d7ac93804d995.

Fix: suppress the warning about dereferencing type-punned pointer that break strict-aliasing rules.

Not implemented. The classes config/Host and dns/HostAddress serve to
different purposes, it is better to keep them separated.

Not implemented. The checksum calculations are different in ICMP and TCP.

Not done and not important, as no bottleneck is noticeable.

Not implemented according to the message:

From: Thomas Jarosch <thomas.jarosch@intra2net.com>
To: guilherme.maciel.ferreira@gmail.com
Subject: Re: DNS library
Date: Mon, 12 Dec 2011 11:12:55 +0100

Hi Guilherme,

On Friday, 9. December 2011 19:24:49 you wrote:
> What do you think is the best approach to include the DNS library in
> the pingcheck? Once it is just headers, do you think I must make the
> DNS library install its header somewhere and search them in the
> pingcheck build process, or do you know a better approach?

I'd say we keep the current approach with the in-tree files: pingcheck is
currently the only project at Intra2net that relies on boost::net::dns.
So building is easier with less external dependencies.

The situation will change as soon as another project uses it, too.

Cheers,
Thomas

Implemented in commit 3625c0e544982518392b842162318db9a9840a1a.

Implemented in 'config' branch.

Implemented in 'test' branch.

Implemented in ipv6 branch.

Implemented in commit ad545958f408012956bcac4cf466c94bec014bcc.

Merge branch 'test'

Conflicts:
src/config/configurationreader.cpp
src/config/configurationreader.h
src/host/messagepayload.cpp
src/link/linkstatus.h
src/main.cpp
src/tcp/tcpheader.cpp

The log level can be configured in the command line, through --log-level=<level> option.