Provide an option to explicitly force IPv4 ping

highvolt-dev / tmo-monitor

A lightweight, cross-platform Python 3 script that can monitor the T-Mobile Home Internet Nokia, Arcadyan, and Sagecom 5G Gateways for 4G/5G bands, cellular site (tower), and internet connectivity and reboots as needed or on-demand.

MIT License

89 stars 15 forks source link

Provide an option to explicitly force IPv4 ping #77

Closed therealbobrice closed 1 year ago

therealbobrice commented 2 years ago

I had issues when my Linux system didn't apparently handle IPv6 pings to Google very well, and for some reason tmo_monitor/gateway/base.py wanted to use IPv6 ping. It would be really nice to either explicitly force ping -4 by default, or allow the user to specify ping -4 as a command line option.

In my configuration I wanted to make double sure it didn't use IPv6 ping, so I just hardcoded base.py to add the -4 flag...

highvolt-dev commented 1 year ago

@therealbobrice I've set the ping command to default to passing the -4 flag except on Mac OS X which does not support it. When using the ipv6 ping option, it will use ping6 or ping -6 as it did before.