search

libwww-perl / LWP-Protocol-https

Provide https support for LWP::UserAgent
https://metacpan.org/pod/LWP::Protocol::https
Other
16 stars 35 forks source link

Enable MultiHomed for IO::Socket::SSL #61

Closed atoomic closed 1 year ago

atoomic commented 3 years ago

Automatically fall back to IPv4 if IPv6 doesn't work

skaji commented 3 years ago

@atoomic Could you explain context for this change in more detail?

For IPv4 and IPv6, I believe IO::Socket::IP should take care of them without MultiHomed argument.

atoomic commented 3 years ago

Using the MultiHomed option for Socket allows to fall back to IPv4 if IPv6 fails.

Some servers have an IPv6 address assigned but lack usable IPv6 networking. Enforcing 'MultiHomed' by default fixes this issue.

note that this can still be disabled if needed by providing MultiHomed => undef to _extra_sock_opts we could also consider enabling it if needed, but IMO using a true value for MultiHomed should be the default behavior

cc: @skaji

oalders commented 1 year ago

Thanks, @atoomic!