]> granicus.if.org Git - python/commitdiff
Fix Issue1368368 - prompt_user_passwd() in FancyURLopener masks 401 Unauthorized...
authorSenthil Kumaran <orsenthil@gmail.com>
Fri, 18 Jun 2010 15:08:18 +0000 (15:08 +0000)
committerSenthil Kumaran <orsenthil@gmail.com>
Fri, 18 Jun 2010 15:08:18 +0000 (15:08 +0000)
Lib/urllib/request.py
Misc/NEWS

index 776053865e44edd12b4d77df791d155770e67e64..15e2dde819a7f362a951cb1af5f5a102ff028029 100644 (file)
@@ -1866,7 +1866,8 @@ class FancyURLopener(URLopener):
         else:
             return self.http_error_default(url, fp, errcode, errmsg, headers)
 
-    def http_error_401(self, url, fp, errcode, errmsg, headers, data=None):
+    def http_error_401(self, url, fp, errcode, errmsg, headers, data=None,
+            retry=False):
         """Error 401 -- authentication required.
         This function supports Basic authentication only."""
         if not 'www-authenticate' in headers:
@@ -1882,13 +1883,17 @@ class FancyURLopener(URLopener):
         if scheme.lower() != 'basic':
             URLopener.http_error_default(self, url, fp,
                                          errcode, errmsg, headers)
+        if not retry:
+            URLopener.http_error_default(self, url, fp, errcode, errmsg,
+                    headers)
         name = 'retry_' + self.type + '_basic_auth'
         if data is None:
             return getattr(self,name)(url, realm)
         else:
             return getattr(self,name)(url, realm, data)
 
-    def http_error_407(self, url, fp, errcode, errmsg, headers, data=None):
+    def http_error_407(self, url, fp, errcode, errmsg, headers, data=None,
+            retry=False):
         """Error 407 -- proxy authentication required.
         This function supports Basic authentication only."""
         if not 'proxy-authenticate' in headers:
@@ -1904,6 +1909,9 @@ class FancyURLopener(URLopener):
         if scheme.lower() != 'basic':
             URLopener.http_error_default(self, url, fp,
                                          errcode, errmsg, headers)
+        if not retry:
+            URLopener.http_error_default(self, url, fp, errcode, errmsg,
+                    headers)
         name = 'retry_proxy_' + self.type + '_basic_auth'
         if data is None:
             return getattr(self,name)(url, realm)
index 7b8bc84125853fc67ffbeb93d476d03452c796b6..a1d3a838f0690fbaace8838fe2ad37ea3dfaf817 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -446,6 +446,10 @@ C-API
 Library
 -------
 
+- Issue #1368368: FancyURLOpener class changed to throw an Exception on wrong
+  password instead of presenting an interactive prompt. Older behavior can be
+  obtained by passing retry=True to http_error_xxx methods of FancyURLOpener.
+
 - Issue #8720: fix regression caused by fix for #4050 by making getsourcefile
   smart enough to find source files in the linecache.