Added the 32 bits version of encode and decode methods

Small code formatting

Asserted the encode16() and decode16() parameters, warning about off-by-one errors

Renamed IcmpMessagePayload to MessagePayload, so it can be used by other messages types

Renamed getters and setters

Moved PingStatus enumeration to its own file, so it can be reused by both Pingers

The Include Guards must match the file's name

Surrounded the receive by a try..catch, so prepare to receive another packet only if the reply isn't a RST

Removing warning about conversion from 'int' to 'uint8_t'

Adding a remark about sequence number from RST

Adding logging to ioctl fails, and replacing the nested if statements by guard clauses (early returns)

Replaced shift operations on IP addresses to an endian-safer approach
- Instead of shifts on sockaddr.sa_data, uses a cast to sockaddr_in.sin_addr.s_addr.
  And as long as the two structures have the same memory layout, the cast is safe.
- Reinterpret four chars (8 bits) as one integer (32 bits), and make it endian safe.

Validates the ifr.ifr_name buffer size before copy (Thomas's advice)

Pinger factory provides only static (class) methods, thus do not require public constructor and destructor.

Set the received status as successful when the RST match what was expected.

Little code format detail

Placed start_receive() method closer to the schedule_timeout_echo_reply(), where it gets called.

Changed the handle_timeout() method to handle_ping_done()
- now it calls the ping call back function
- and do not schedule the next ping anymore, let it to the callback responsability

Renamed create_tcp_header() method to create_ack_request()

Move the ACK segment send functionality to the send_ack_request() method

Split the method, separated some functionality in schedule_timeout_rst_reply()

Renamed timeout variable to the context of TCP instead of ICMP

Sequence number does not identify a particular segment, it is always the same

Missing header files and identation

Retrieve the local IP address from the interface using the low level ioctl, plus some code enhancements

Added and formatted some error messages

Split methods
- create_tcp_header() to create the TCP header for ACK ping
- print_rst_reply() to print the TCP header for RST reply

Cosmetic changes
- improved variable names
- added "Warning" to some logs

Removed commentted code

Formatting log messages

Working TCP pinger
- Needs to fix the source IP address used in the pseudo IP header

Split the TcpHeader implementation and header

Removed the unused get_identifier() method

Small code formatting

Fixed socket typedef, it is a raw socket, not stream socket

Added the default protocol to use in the config file

Better comment description

Fixed the return statement, the method has the return statement in the correct place

Fully functional pinger factory

Added the remaining TCP pinger arguments
- still not working, but the skeleton is done

Code formatting

Solving some conversion and variable shadow warnings

Adding the ping protocol configuration option

Moving the PingProtocol enumeration from PingerFactory to its own file
- Added a method to convert from string to the enumeration

The "using namespace std" exempt the use of "std::" qualifier

Note to myself: rework the checksum to use it also in the TCP stuff

Fixed include guards name

Bring aboard the TCP header and support classes
- Tested with a TCP ping prototype, solved the checksum issue
- Needs to be polished

Bring aboard a aternative version of Boost's TCP protocol
- This version allows low level socket manipulation of TCP segments, whereas the SOCK_STREAM doesn't allow.

Moved the IP classes to its own directory.

Comment to avoid warning

Calling set_ping_status() in ping function and added some documentation

Bring aboard the Ping factory

Using inheritance to commute between pingers

Modified configuration file

Renamed encode/decode to encode16/decode16
- specify the number of bits it encodes/decodes
- planning support for encode/decode of 32 bits used by TCP header

Replaced the assert by a conditional
- because the amount of data received can differ from the amount we expect, but this cannot make the application to abort.

Bring aboard TCP pinger class

Moved the icmppinger from host to icmp folder

Included a precondition to DnsResolver::handle_ip_address()

Added a tip in the Readme

Renamed BoostPinger to IcmpPinger
- the TCP pinger will be another class to avoid overloading this one with TCP stuff
- the name looks better =)

Minimum version required of Boost is 1.44

First public commit

Don't package files from kdevelop

Started PC-Lint settings file

Improve packaging: Include correct license files

Add GPLv2 + linking exception to the source

Remove t2n call stuff

Add boost license + proper copyright were needed

Update TODO list

Properly handle ICMP packet replies: The kernel sends us a copy of all incoming ICMP messages on every ICMP socket

Turn RepliesCount into ReceivedReply flag

Update TODO file

Added doxygen comments in the most important places.

Turn BoostPinger into true async operation

No rescheduling in BoostPinger

Implemented the feature:
- Make the link down interval also configurable. Today it is possible just to
  configure the amount of time the link must to be up to notify. Do another
  configuration to allow the time of link down.

Two TODO entrys solved

Reuse BoostPinger object

Fix ping of IP addresses (don't do DNS lookup)

Solved the item:
- Perform a better treatment of missing configuration items, actually the
  application just crashes.

All items that are strictly required have default values, or are checked against
valid values. The last item missing was nameserver, which has a default value now.

Alternative approach to solve:
- If one of the "[host]" in the configuration file omit the "interval" entry,
  the application uses the next "[host]"'s "interval" entry, instead of a
  default value. The problem is specific to the configuration read classes
  that are not smart enough to verify which "[host]" have this entry and which
  don't.
  Example:
     [host]
     name=A
     [host]
     name=B
     interval=1
     [host]
     name=C
     interval=3
  Thus, the tuples will be (A,1) , (B,3), (C,?), but A should have a default
  value, instead of borrow B's value.

The approach consists of ensure the program only runs if there is an interval
for every single host.

Allowing unknown options and changed the default host down limit to 0

Clean thread shutdown

Remove boost signal handler: It's overdesigned for our use case and creates another thread just to wait for signals

Handle signals in a thread safe way

Convert signal handler library to unix line endings

Create unique identifier for every ping

Moved the boost signal handler library

Updated TODO

The multi-threading approach solves the blocking

Merge branch 'pingcheck' of ssh://aqua.m.i2n/data/git/intranator/source into pingcheck

TODO changes

working signal handler

Renamed project from libpingcheck to pingcheck

Update TODO

Modularization of the main.cpp
- semantic order enforced by parameters
- less spaghetti code
- code more readable
- can be moved to a class more easily

Defining the LinkStatusAnalyzerItem
- to keep the standard
- to provide the same benefits as the other Item types

Defining the type PingSchedulerList as a vector of PingSchedulerItem
- to keep the standard
- same benefits as HostList