]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-deploy.git/commit
Support hostname that resolve to IPv6-only address
authorFrode Nordahl <frode@nordahl.net>
Thu, 27 Mar 2014 08:20:52 +0000 (09:20 +0100)
committerFrode Nordahl <frode@nordahl.net>
Thu, 27 Mar 2014 08:20:52 +0000 (09:20 +0100)
commita09cf17583ca558d3e4c77a1682ed01c223f182d
treecf5421687fed0edef123c1c461fa5f1717cecdb1
parent1a6661515446cd76899310624e84238ea71c7ef6
Support hostname that resolve to IPv6-only address

The current hostname validation does not cope with IPv6-only hostnames. Use getaddrinfo instead of gethostbyname to fix this. getaddrinfo raises the same exceptions and should work like a drop-in-replacement in this scenario.

We should also address the IPv4-only check for if then input is an IP-address but the use of split on ':' is problematic and prevents code to check for IPv6-addresses as input. I'm not sure what the thought behind allowing "name:host" is so leaving it untouched.

I have proposed code to check and warn for both IPv4 and IPv6 address input that I can provide as soon as I understand the scope of parsing "name:host" input.
ceph_deploy/util/arg_validators.py