Improved hostname verification

pull/1243/head
Chris Caron 6 days ago
parent f37dfbf5a1
commit eed92255c4

@ -325,7 +325,7 @@ def is_hostname(hostname, ipv4=True, ipv6=True, underscore=True):
labels = hostname.split(".") labels = hostname.split(".")
# ipv4 check # ipv4 check
if len(labels) == 4 and re.match(r'[0-9.]+', hostname): if len(labels) == 4 and re.match(r'^[0-9.]+$', hostname):
return is_ipaddr(hostname, ipv4=ipv4, ipv6=False) return is_ipaddr(hostname, ipv4=ipv4, ipv6=False)
# - RFC 1123 permits hostname labels to start with digits # - RFC 1123 permits hostname labels to start with digits

@ -87,6 +87,22 @@ def test_parse_url_general():
assert result['qsd+'] == {} assert result['qsd+'] == {}
assert result['qsd:'] == {} assert result['qsd:'] == {}
# GitHub Ticket 1234 - Unparseable Hostname
result = utils.parse_url('http://5t4m59hl-34343.euw.devtunnels.ms')
assert result['schema'] == 'http'
assert result['host'] == '5t4m59hl-34343.euw.devtunnels.ms'
assert result['port'] is None
assert result['user'] is None
assert result['password'] is None
assert result['fullpath'] is None
assert result['path'] is None
assert result['query'] is None
assert result['url'] == 'http://5t4m59hl-34343.euw.devtunnels.ms'
assert result['qsd'] == {}
assert result['qsd-'] == {}
assert result['qsd+'] == {}
assert result['qsd:'] == {}
result = utils.parse_url('http://hostname/') result = utils.parse_url('http://hostname/')
assert result['schema'] == 'http' assert result['schema'] == 'http'
assert result['host'] == 'hostname' assert result['host'] == 'hostname'

Loading…
Cancel
Save