made nicer static variables of packet dump mode and location

increased version to 0.4 revision 0: new DNS, merged Scheduler with Rotate and made it use async DNS

remove the footer saying that vim is the best editor -- anyone knows anyway

created arg recursion_count to async_resolve and many other to avoid infinite loops

* arg recursion_count replaces cname_count which up to now identified problems in post-processing
* created simple loop test
* added option to configure recursion depth limit to DnsMaster

minor other changes:
* corrected docu for resolved-ip-ttl-threshold option: is not milliseconds but seconds
* add test to ResolverBase: check number of callbacks and warn if suspiciously many

added option min-time-between-resolves-option and tests for it

define this as revision 3 of version 0.3 and add to intranator source master

had forgotten to unset debug option (max number of acceptable errors was 1)

did minor changes; used this version for testing over week-end

fixed bug that caused outdated IPsto be returned from cache; added test for that

found reason for 0.0.0.0 IPs in logs: IcmpPingers are created without IP but register immediately with PacketDistributor

adjust log message, should probably remove debug output some time

added option log-file and option FILE to log-output; use in main and adjusted unit tests

tested new DNS with internal server, make more robust against caching; works nicely now

added test to DNS: load cache from file

fixed bug in TimeToLive: get huge TTLs from cast to uint32_t of negative values

fixed 2 bugs and made clearer that Long-term timer in DnsResolver is not affected by finalize_resolve

bugs:
* HostAddress::is_valid returned opposite
* use the returned IP to ping and do not request another in PingScheduler::ping
* move reset of ContinueOnOutdatedIp within PingScheduler to after ping

long-term timer:
* added a variable DnsResolver: LongtermTimerIsActive
* added a function to ResolverBase: is_waiting_to_resolve
* added a function cancel_resolve to PingScheduler that decides whether to cancel or not
* added an arg to DnsResolver::stop_trying

removed the HostStatus::report_dns_resolution_failure (not used any more)

ensured LogPrefix is used in DNS and PingScheduler; shortened lines; remove vim end note

more cname-skip unit-tests and simplification of skip-finder function

created and passed first unit tests for DNS; finished recovery from PingScheduler::ContinueOnOutdatedIPs

Also:
* renamed PingScheduler::try_to_ping to PingScheduler::ping_when_ready
* fixed string constant DnsCache::DoNotUseCacheFile
* moved Cname to own header file
* created constructor of DnsMaster with DnsCacheItem so can test cache offline
* fixed retrieval of outdated CName

remote PingRotate

merged PingRotate into PingScheduler; fixed save/load of cache to/from file; started creating DNS unittest

code now compiles and links and pings again!

merged because through Asynchronicity it is more difficult to give feedback from Rotate to Scheduler
further changes
* in cache ensure that there is not cname and ip for same hostname
* in cache remove trailing dots from hostnames
* by default ask only for [number of] updatd IPs in retrieval from resolver
* remove some left-overs from recursive resolving (e.g. Recursor creation in master)

simplified dns (no self-made recursion); merge PingScheduler and PingRotate; make them use Async DNS

does not compile yet, work in progress!

finished self-implementation of DNS resolver recursion; will now remove all that!

implemented resolving that avoids caching by creating non-recursive queries
that are directly talking to different name servers; will remove this now
because firewalls might prevent direct communication and do not want to
self-implement dnssec some time.
--> now remove own implementation of recursion in resolving and try to deal
    with cached results

continue implementation; first tests with recursion returned IPs but then added cancel code

added option to cancel Recursor if timeout is reached
sometimes re-try resolving more quickly instead of calling handle_unavailable
changed saving of ttls: save original value and creation time by adding friend class HostAddress
cache can do the recursive ip retrieval, better place than ResolverBase

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