icmp echo request and response automatically adds 8 bytes to the payload with a tick count about indy HOT 3 OPEN

The tick counter is put in the payload to help calculate a value for the ReplyStatus.MsRoundTripTime property when the echo response comes back. I suppose the internal FStartTime member could be used for that purpose instead.

Allowing users to send a binary payload instead of a string payload is a good idea, especially since it doesn't appear the client returns that response data back to the caller anyway.

from indy.

rlebeau, thanks for your comments. We just re-wrote it internally to do what we needed to do.

from indy.

I updated TIdIcmpClient to support TIdBytes for user-defined data. I have not removed the tick counter from the ICMP payload yet, though.

from indy.