changed how dns deals with cnames and recursion: remember cnames and implement recursive lookup here using DnsResolvers that are not saved in DnsMaster.

Experiments with own name server and recursive cnames showed that TTLs are not always the minimum.
This way we have better control about resulting TTLs
and debugging is easier since DNS caching is avoided.

Also create unique id for each dns message and check reply
and in main warn if using debug option max_exceptions.

started experimenting with DnsResolver; does send stuff but has not received anything yet

moved new dns code from temp dir to dns subdir; changed serialization; added destructors; added boost libs --> can link now!

moved new dns code from temp dir into proper dns dir

deleted old DNS resolver and its factory

adjusted rest of pingchecker to use new dns in very trivial fasion; compiles but need to re-design async calls in scheduler

added dns cache file option

added missing functions and config vars used by pingscheduler/pingrotate

* functions have_up_to_date_ips and get_resolved_ip_count to resolvers
* limitation to only IPv4/v6 to DnsResolver
* --> DnsMaster remembers resolvers by pair(host name, ip version)
* DnsMaster holds global config vars NameServer, ResolvedIpTtlThreshold and MaxAddressResolutionAttempts
  and provides public getters for them

implemented serialization, is compiling

done except for saving/loading cache; not being used yet but does compile (except for linker error caused by old dns system still in CMakeLists.txt)

continuing with dns rewrite; today realized that cache is better as own class and ResolverBase gets more powerfull

added first half-ready versions of new DNS files in temp directory dns_neww to avoid overwriting of old reference code

renamed expired_resolved_ip to have_up_to_date_ip because that makes clearer what it is used for

re-designed DNS plan because with first approach all class variables would be maps: hostname --> actual_var

do actually rotate protocols in PingRotate

moved try-catch around icmp packet parsing from IcmpPacketDistributor to IcmpPacketFactory so can dump when exception is thrown

changed start of pings to time when io_service.run() is called (and not earlier)

made PingRotate actually rotate through protocols, and not just stick with last protocol

updated revision (version now 0.3.2) and fixed creation of version string

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