Search from the end of the host/port combination to find the colon which

Skip Montanaro · Skip Montanaro · commit 10e6e0e61a38 · 2004-09-14T16:32:02.000Z
separates ip address from the port to accommodate ipv6 addresses.
diff --git a/Lib/httplib.py b/Lib/httplib.py
@@ -524,7 +524,7 @@ def __init__(self, host, port=None, strict=None):
 
     def _set_hostport(self, host, port):
         if port is None:
-            i = host.find(':')
+            i = host.rfind(':')
             if i >= 0:
                 try:
                     port = int(host[i+1:])
diff --git a/Lib/test/test_httplib.py b/Lib/test/test_httplib.py
@@ -118,6 +118,13 @@ def _test():
         else:
             print "Expect InvalidURL"
 
+    for hp in ("[fe80::207:e9ff:fe9b]:8000", "www.python.org:80",
+               "www.python.org"):
+        try:
+            h = httplib.HTTP(hp)
+        except httplib.InvalidURL:
+            print "InvalidURL raised erroneously"
+
     # test response with multiple message headers with the same field name.
     text = ('HTTP/1.1 200 OK\r\n'
             'Set-Cookie: Customer="WILE_E_COYOTE"; Version="1"; Path="/acme"\r\n'
