[Python-checkins] r59661 - in python/trunk/Lib: test/test_urllib.py urllib.py
kurt.kaiser
python-checkins at python.org
Wed Jan 2 05:11:29 CET 2008
Author: kurt.kaiser
Date: Wed Jan 2 05:11:28 2008
New Revision: 59661
Modified:
python/trunk/Lib/test/test_urllib.py
python/trunk/Lib/urllib.py
Log:
Issue1177
r58207 and r58247 patch logic is reversed. I noticed this when I
tried to use urllib to retrieve a file which required auth.
Fix that and add a test for 401 error to verify.
Modified: python/trunk/Lib/test/test_urllib.py
==============================================================================
--- python/trunk/Lib/test/test_urllib.py (original)
+++ python/trunk/Lib/test/test_urllib.py Wed Jan 2 05:11:28 2008
@@ -126,6 +126,19 @@
finally:
self.unfakehttp()
+ def test_read_bogus(self):
+ "urlopen() should raise IOError for many error codes."
+ self.fakehttp('''HTTP/1.1 401 Authentication Required
+Date: Wed, 02 Jan 2008 03:03:54 GMT
+Server: Apache/1.3.33 (Debian GNU/Linux) mod_ssl/2.8.22 OpenSSL/0.9.7e
+Connection: close
+Content-Type: text/html; charset=iso-8859-1
+''')
+ try:
+ self.assertRaises(IOError, urllib.urlopen, "http://python.org/")
+ finally:
+ self.unfakehttp()
+
def test_empty_socket(self):
"""urlopen() raises IOError if the underlying socket does not send any
data. (#1680230) """
Modified: python/trunk/Lib/urllib.py
==============================================================================
--- python/trunk/Lib/urllib.py (original)
+++ python/trunk/Lib/urllib.py Wed Jan 2 05:11:28 2008
@@ -342,7 +342,7 @@
'got a bad status line', None)
# According to RFC 2616, "2xx" code indicates that the client's
# request was successfully received, understood, and accepted.
- if not (200 <= errcode < 300):
+ if (200 <= errcode < 300):
return addinfourl(fp, headers, "http:" + url)
else:
if data is None:
@@ -437,7 +437,7 @@
'got a bad status line', None)
# According to RFC 2616, "2xx" code indicates that the client's
# request was successfully received, understood, and accepted.
- if not (200 <= errcode < 300):
+ if (200 <= errcode < 300):
return addinfourl(fp, headers, "https:" + url)
else:
if data is None:
More information about the Python-checkins
mailing list