update Readme, TODO and example pingcheck.conf

fixed trouble in unittests: when I changed/added options I forgot to adjust corresponding unittests

fixed same bug as for DestinationUnreachable in TimeExceeded messages: accept reply from mid-way

altered unit test accordingly
removed unnecessary implementations of match_xy functions that also just return false

fixed bug in match_destination_unreachable: message does not have to come from original destination

that means we have to interpret the ip header within the icmp data, which is done by converting
IcmpData::raw_data to a stream which is fed to IpvXHeader.read
Also means that address to test for has to be forwarded to IcmpDestinationUnreachableData
and cannot be simply tested in IcmpData already --> change function prototype in all IcmpData classes

added more tests, fixed trouble with position of test data relative to build dir

can now interpret pcaps from tcpdump; created own pcap.h and pcap.cpp because is needed in 3 places now:

* icmp/icmppacketfactory
* tools/feed_packet_data
* tests/test_icmppacket

pcaps from tcpdump have network_type = 1, so include a interface struct in packet data before actual data.
Our own dumps were network_type=101 and did not include that interface info

added more testing pcap data

created data directory in test directory for pcap files for IcmpPacket tests

added revision to version string (0.3 --> 0.3.1) to more easily see if testing the correct version (e.g. if patching tarballs)

also added build date to output of pingcheck --version

added more getter to IcmpPacket so can test more

added new unittest for IcmpPacket

fixed bugs that caused DestinationUnreachable / TimeExceeded messages to NEVER match anything

also added function to get ttl of original request IP packet but unfortunately that always returns 1 if time exceeded :-(
Also improved error reporting in ICMP packet parsing process

added option to print raw message data in text form for debugging

added code to parse ICMP type TimeExceeded

* format is identical to DestinationUnreachable and other ICMP replies
  --> created a common half-abstract base class IcmpData_PingFailReply
* added a match_time_exceeded to IcmpPacket which is called by IcmpPinger

in case of unknown ICMP reply type, consume data instead of returning failure in IcmpPacket::read

added ip source and destination address to IpHeader::to_string

increased version to 0.3 since have now new ICMP packet structure and
packet feeder tool and will create new tar from this

added a simple text output (to_string) for DestinationUnreachable ICMP data

added a count of good/bad packets to feed_packet_data tool

added rule to create feed_packet tool to CMakeLists.txt

check if file is really open after opening; changed return codes and log levels

got the tool feed_packet_data to compile and run and do reasonable stuff

added a dump_option to IcmpPacketFactory creation function; better error message if temp file creation fails

started creating a utility program in new folder tools that feeds packet data directly into ICMP/TCP factories

moved remaining implementations in ipv6header.h to cpp; avoid conversion warnings

give credit to Christopher Kohlhoff (boost asio) and Guilherme Ferreira

consequently moved all implementation from h to cpp files

fixed bug in reading of checksum (which was found using the unittests -- great thing!)

updated icmp unittests

moved some implementations from h to cpp files because of linker trouble

now working again: creating echo requests and parsing replies

also:
* created a to_string method for easier output
* in constructors of IP headers set version right away
* compare that version to the one received in IcmpPacket::read

re-ordered class for IcmpPacket

old: IcmpPacket has and IpvXHeader, an IcmpvXHeader which varies according to ICMP message type and also contains part of Icmp message data, and an IcmpData which is a string;
new: IcmpPacket has an IpHeader, an IcmpHeader which represents only the common first 4 byte of ICMP v4/v6, and IcmpData which is either a IcmpEchoData for echo request/reply or a IcmpDestinationUnreachableData

tried to abstract away as much as possible (for a lay programmer like me) the difference between v4 and v6

This code now compiles, need to see whether it does anything usefull

Tests not compiled/run yet

added encode_string to MessagePayload; removed checksum and icmpchecksum since functionality moved into MessagePayload, IcmpHeader and IcmpData

created abstract base class IpHeader of Ipv4Header and Ipv6Header

added debug option enforcing dump of ICMP packets even if not broken

fixed bug: dumping packets leaked char array (dump file name) on heap

dumping now into a .pcap file with nicer filename, so can analyze with wireshark

dumping package data works

started copying icmp data before parsing it and dumping it into temp files if parsing fails

does compile but causes problems in data parsing

make check for end of loop in icmppacketdistributor more understandable (|= --> =)

Check weak_ptr before using it

No need for intermediate weak_ptr

Use const reference for smart_ptr in BOOST_FOREACH

Move graph documentation to doc/

fix bug that caused crash for broken packets; change order of vars to avoid compiler warnings; minor change in debug output

updated TODO

removed another debugging test and corrected one debuggin output

created a graph to show how IcmpPinger, IcmpPacketDistributor and other important classes interact

created using yEd 3.12.2

give up trying to define IcmpPacketDistributor in own files, just use icmppinger.h / .cpp for it

The two classes depend on each other, so none compiles without the other
  if they are defined in separate files. In the same file, the compiler
  manages.

got packetdistributor to work, but compiles/links only with packetdistributor inside icmppinger.h/cpp

limit line length of new files to 80 characters

use new file name (icmppaKetdistributor --> icmppacCKetdistributor) in includes and macros

also rename file in CMakeLists.txt

rename IcmpPaketDistributor files to IcmpPacketDistributor (k --> ck)

start removing socket use from IcmpPinger to only have IcmpPacketDistributor deal with raw data

clean up IcmpPaketDistributors after stopping pingers

Pinger holds weak_ptr to self, Pingers created through static create functions; IcmpPinger::create registers with IcmpPaketDistributor

started to implement an icmp paket distributor; compiles and works at first, but gives segmentation fault at exit

added a test for pingcheck version to avoid trouble with IP parsing in 'older' pingcheckers

another amendment to signal handler: make signaled_flags_XY volatile sig_atomic_t (I was ignorant)

added saving of history and results and function to load and print stats about them

added a little failsave test into tester; simplified logging of exceptions; recognize unexpected online states and warn; removed pre-logging Tester._print

had forgotten to reset default pingcheck binary from testing to production

amendment to last commit: only makes sense if use different flags per signal type

removed logging from signal handler; create one handler per signal type

changed return type of get_configuration in main: return configuration and bool flag

this way, always get a valid configuration object with default values and some values set
  even if parsing other options failed. Important for --version where usually fail to read
  config file but do not need it anyway

changed treatment of --version arg; remember to print it in Configuration like --daemon option

corrected type of TTL: is now uint32_t everywhere (except for uint8_t in Payload data)

removed Readme.code because was neither up to date nor informative; updated TODO

raised version number from 0.1 to 0.2

remember pings based on host, not IP; correct logging logic in main loop

for host-ip binding, needed to slightly change output from dnsresolver

treat output as "left-over" only after terminatin process; limit number of lines

removed assert that does not make sense and is often violated

(IPs are not updated very often, so can easily happen that an IP's TTL was reached.
 It is easier to ignore these cases than to avoid them, IPs will with next test be updated.)

had forgotten to disable testing setting in pingcheck main: limited the max number of exceptions ignore

done with long_term_test.py

* set timeout for ping reply -- very important variable!
* try to get more output when pingcheck process is stopped/dying
* added and commented out sigchld / child signal handler
* different logic how state changes are set
* use ConndState
* lots of other changes...

improved tell-connd state representation

* use constants for online states and subsystems
* add function to output complete state
* deal with exceptions from running binary

Raised importance of log output in signal handler: info for most, notice for term/kill

use logging for all output, improve understanding of connd, improved process termination, simplify code

some minor doc change I had overlooked

added a test python script that goes online/offline and checks if pingers pinged enough

renamed IcmpPinger::handle_ping_done to handle_timeout; handle error_codes from timer async_wait

some whitespace change I had forgotten

added case for unexpected signals in signal handler

make error return codes from main unique again

introduce a max number of exceptions ignored in io_service main loop

mainly for testing and debugging

properly initialize variables remembering original signal handlers; re-set only if were set

continue waiting for data if receive handler received an error code

2 reasons:
* in other error cases do the same
* there is a timer running in the background which will send the next ping after some time

do not wait for IcmpEchoReply if sending of IcmpEchoRequest has failed completely

update docu messages for io_service

Clean up output, whitespace,

* there were several cases of GlobalLogger.error() << "Error: ..." which is redundant
* there was one instance of streaming to cerr instead of GlobalLogger.error
* remove trailing whitespace

minor change in logging: avoid duplicated "Error: "

new signal handling in main; removed all thread-related vars, use same IoServiceItem instead

temp commit before leaving, started to change signal handling and use singel io_service

hotfix for problem with 0-packages

* added flag ReceiveHandlerInPlace to icmppinger
* to be sure, also check if reply was received already in handle_receive_icmp_packet
* in handle_receive_icmp_packet also check error code
* in icmp package read, print strings for failure codes
* prepend IP to pinger log output so can trace which thread did what

changed logic of icmp package read/write and operator>> / operator<<

* read and write are implemented on their own, >> and << call them
  (not the other way round as before)
* read returns not only a bool but a more informative flag
* moved operator>> and << from Icmpv4/6packet to base class IcmpPacket
  since they are identical

fixed parsing of icmp data: only parses right amount of data

fixed possible reason for broken icmp packages: restricted read of icmp payload data; also moved def of buffer size from h to cpp

fixed possible reason for broken icmp packages: restricted read of icmp payload data; also moved def of buffer size from h to cpp

replace magic number 64k with constant SOCKET_BUFFER_SIZE

improved logging output (once again)