tcpping leaks 255 ttl

Question

tcpping leaks 255 ttl

dmascord opened this issue 3 years ago · 9 comments

Expected Behavior

tcpping should show packet loss when ttl is expired:

tcpping -C -x 5 100.100.100.199
100.100.100.199 : - - - - -

Current Behavior

tcpping responds with 255, and smokeping interprets as 255ms

tcpping -C -x 5 100.100.100.199
100.100.100.199 : 255 255 255 255 255

Steps to Reproduce

Configure smokeping to use tcpping Probes

dmascord commented 3 years ago

OK...

Answer 1 · 2021-10-11T22:24:11.000Z

Thanks for opening your first issue here! Be sure to follow the bug or feature issue templates!

Answer 2 · 2021-10-12T00:08:29.000Z

I have modified the latest tcpping version to the latest version, which handles this scenario properly, but the newest version depends on traceroute -M, which your busybox version does not yet support.

I have updated tcpping to fallback to tcptraceroute if traceroute does not support -M at https://raw.githubusercontent.com/dmascord/tcpping/add-tcptraceroute-fallback/tcpping

Answer 3 · 2021-10-12T23:42:39.000Z

There is still a problem with the tcptraceroute that is included in this image. It is setting the window size to 0, which seems to have firewalls dropping the packets. traceroute -M tcp works well (but the busybox traceroute doesn't seem to have this option), but tcptraceroute, even with the updated tcpping fails on some destinations for SYN and ACK modes due to the underlyung tcptraceroute issue

Answer 4 · 2021-10-13T00:18:12.000Z

OK, found out why tcptraceroute isn't working properly for some endpoints, which is due to the missing TCP Options.

This has been addressed in mct/tcptraceroute#14

How do we get this updated tcptraceroute and associated tcpping into a smokeping docker image ?

Answer 5 · 2021-11-12T02:11:56.000Z

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Answer 6 · 2021-11-12T02:45:03.000Z

Stale? This is still an issue

Answer 7 · 2021-12-13T02:17:02.000Z

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Answer 8 · 2022-01-13T02:26:10.000Z

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.