librespot-org
Expose service to both IPv4 and IPv6
Change to listen to [::0] given this will make the application listen on both ipv4 and ipv6. This is useful to enable support for ipv6 only networks. Given most users will be on v4 only or dual stacked networks it is of course important to make sure that works, in my testing i have not found any case where this change created a problem for those devices.
in my testing i have not found any case where this change created a problem for those devices.
Just so it's clear, what patforms did you test?
"Technically", I guess this would fail on platforms where the IPv6 stack doesn't exist (i.e. if you compile the kernel by explicitly disabling IPv6, which I guess nobody does unless you want to torture yourself). But I guess the number of people doing that is way smaller that the people that want/need IPv6 (i.e. those using NAT64, etc..)
Looks good to me. Can you add an entry to the changelog? Then we can merge.
This doesn't work for IPv4 on Windows :( Dual-stack sockets aren't the default.
Perhaps best to use Ipv6Addr::UNSPECIFIED.into() only on non-windows?
Looks like you're right: https://github.com/rust-lang/rfcs/issues/957
That's really interesting actually
We're aiming for a 0.5 release next week. Would be great to get this in, if you could do the last touch-up. Thanks.