summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSenthil Kumaran <orsenthil@gmail.com>2010-06-01 12:53:48 (GMT)
committerSenthil Kumaran <orsenthil@gmail.com>2010-06-01 12:53:48 (GMT)
commitf4998acb5ab3c2fbbcb24e84ed7ed8721ab20f5c (patch)
tree4438569721063d0d4872a78c8ac1e5e39acc2d1e
parentd35251dc19b6f2d1cacbb622a578c81f33d8553a (diff)
downloadcpython-f4998acb5ab3c2fbbcb24e84ed7ed8721ab20f5c.zip
cpython-f4998acb5ab3c2fbbcb24e84ed7ed8721ab20f5c.tar.gz
cpython-f4998acb5ab3c2fbbcb24e84ed7ed8721ab20f5c.tar.bz2
Merged revisions 81636 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk ........ r81636 | senthil.kumaran | 2010-06-01 18:10:07 +0530 (Tue, 01 Jun 2010) | 3 lines Fix Issue8797 - urllib2 basic authentication fix for wrong passwords. It fails after 5 retries. ........
-rw-r--r--Lib/test/test_urllib2.py1
-rw-r--r--Lib/urllib/request.py9
2 files changed, 9 insertions, 1 deletions
diff --git a/Lib/test/test_urllib2.py b/Lib/test/test_urllib2.py
index f173aa0..080daa4 100644
--- a/Lib/test/test_urllib2.py
+++ b/Lib/test/test_urllib2.py
@@ -1151,7 +1151,6 @@ class HandlerTests(unittest.TestCase):
self.assertEqual(len(http_handler.requests), 1)
self.assertFalse(http_handler.requests[0].has_header(auth_header))
-
class MiscTests(unittest.TestCase):
def test_build_opener(self):
diff --git a/Lib/urllib/request.py b/Lib/urllib/request.py
index c912833..7760538 100644
--- a/Lib/urllib/request.py
+++ b/Lib/urllib/request.py
@@ -775,12 +775,21 @@ class AbstractBasicAuthHandler:
password_mgr = HTTPPasswordMgr()
self.passwd = password_mgr
self.add_password = self.passwd.add_password
+ self.retried = 0
def http_error_auth_reqed(self, authreq, host, req, headers):
# host may be an authority (without userinfo) or a URL with an
# authority
# XXX could be multiple headers
authreq = headers.get(authreq, None)
+
+ if self.retried > 5:
+ # retry sending the username:password 5 times before failing.
+ raise HTTPError(req.get_full_url(), 401, "basic auth failed",
+ headers, None)
+ else:
+ self.retried += 1
+
if authreq:
mo = AbstractBasicAuthHandler.rx.search(authreq)
if mo: