http: strip credentials from urllib2 manager URIs (issue2885)

urllib2 password manager does not strip credentials from URIs registered with
add_password() and compare them with stripped URIs in find_password(). Remove
credentials from URIs returned by util.url.authinfo(). It sometimes works when
no port was specified as the URI host is registered too.

mercurial/util.py

@@ -1565,7 +1565,9 b' class url(object):'
             self.user, self.passwd = user, passwd
         if not self.user:
             return (s, None)
-        return (s, (None, (str(self), self.host),
+        # authinfo[1] is passed to urllib2 password manager, and its URIs
+        # must not contain credentials.
+        return (s, (None, (s, self.host),
                     self.user, self.passwd or ''))
 
     def isabs(self):

tests/test-hgweb-auth.py

@@ -1,4 +1,5 b''
 from mercurial import demandimport; demandimport.enable()
+import urllib2
 from mercurial import ui, util
 from mercurial import url
 from mercurial.error import Abort
@@ -95,3 +96,12 b" test({'x.prefix': 'http://example.org/fo"
       'y.username': 'y',
       'y.password': 'ypassword'},
      urls=['http://[e-mail]/foo/bar'])
+
+def testauthinfo(fullurl, authurl):
+    print 'URIs:', fullurl, authurl
+    pm = urllib2.HTTPPasswordMgrWithDefaultRealm()
+    pm.add_password(*util.url(fullurl).authinfo()[1])
+    print pm.find_user_password('test', authurl)
+
+print '\n*** Test urllib2 and util.url\n'
+testauthinfo('http://[e-mail]:8080/foo', 'http://example.com:8080/foo')

tests/test-hgweb-auth.py.out

@@ -189,3 +189,8 b' URI: http://[e-mail]/foo'
 CFG: {x.password: xpassword, x.prefix: http://example.org/foo/bar, x.username: None, y.password: ypassword, y.prefix: http://example.org/foo, y.username: y}
 URI: http://[e-mail]/foo/bar
      ('y', 'xpassword')
+
+*** Test urllib2 and util.url
+
+URIs: http://[e-mail]:8080/foo http://example.com:8080/foo
+('user', '')