Feature request: support for IPv6

[hugoh]

IPv4 addresses are supported at ping hosts (e.g., 8.8.8.8), but IPv6 (e.g., 2001:4860:4860::8888) isn't on Linux as it requires the use of ping6 instead of ping.

It would be nice to support both.

[highvolt-dev]

Thanks for the request, @hugoh

Should be able to implement by passing the the -6 flag to the underlying ping command for Windows and Linux but we'll need ping6 for OS X.

[highvolt-dev]

@hugoh this was implemented by #61

[hugoh]

Awesome, thank you!

…

On Feb 6, 2022, at 5:00 PM, highvolt-dev ***@***.***> wrote:  @hugoh this was implemented by #61 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.

[highvolt-dev]

Reopened based on feedback from @hugoh in #61

[hugoh]

Thanks. It's working except for 1 aspect: I could not figure out how to enable it via .env. I tried to set tmo_6, tmo_ping6, tmo_ping_6, tmo_ping-6 to True without success. Am I missing something?

[highvolt-dev]

It's intended to be set via tmo_ping_6=True

Can you verify the rest of your .env file is being parsed?

After restructuring the repo into modules and to use setup.py, it looks like the .env doesn't get picked up because it installs a copy of the program rather than using your cloned repo.

Try reinstalling with pip3 install -e . so it installs with a symlink to the repo directory or installing into a virtualenv that is a subdirectory of the repo directory so it can traverse up to get to the .env file

[hugoh]

It doesn't pick up the tmo_ping_6 option.

The following .env:

tmo_ping_host=github.com
tmo_ping_6=True
tmo_skip_reboot=True
tmo_print_config=True

gives the following config:

Script configuration:
[..]
  Ping configuration:
    Host: github.com
    Count: 1
    Interval: 10
    Protocol: IPv4

It picks up tmo_ping_host but not tmo_ping_6.

[highvolt-dev]

Thanks for the sanity check, @hugoh - just committed what should be the necessary fix