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)

added a handler for failed BOOST_ASSERT-ions; add global try-catch in main to catch exceptions

added missing std::, commit before last does not compile without it ...  sorry!

debug-log-message when there is unused data in buffer, which is probably reason for frequent broken packages

added global try-catch to main

made text log output slightly more informative and correct

added target "dist" to CMakeLists.txt, which uses "git archive" to build source tarball

more rigorous command line option checking; ensure parser errors are printed to stderr

warn in case of unrecognized command line options; added more info output concerning log output and level

had forgotten to remove a (rather stupid) debug test line

converted 3 constants into configuration variables, removing corresponding TODOs; minor change in variable orders to make compiler happier

make abstract base classes boost::noncopyable to avoid warnings concerning copy-assignment and -constructors

use new function name of new log level string function from libi2ncommon; had overlooked one debugging output

added /usr/intranator prefix to spec file

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