There is no associated test?

Merging now.

Apparently@glynos had a comment.

There's really no way to test this currently as far as I can tell, at least without running a server which listens on an ipv6 address.

@glynos Can we merge it now first, then make sure this is tested at a later date when we're refactoring the implementations? I'd be happy to do it, but I think just addressing the issue right now this way should work.

Thanks@glynos!