diff options
author | Kurt B. Kaiser <kbk@shore.net> | 2008-01-02 04:11:28 (GMT) |
---|---|---|
committer | Kurt B. Kaiser <kbk@shore.net> | 2008-01-02 04:11:28 (GMT) |
commit | 0f7c25d20f68f5fc7f51e05161b0eeb34b8050de (patch) | |
tree | 14d223c501ea9deb94c92597a5229b9d21c8b4e7 /Lib/urllib.py | |
parent | 9fd2bcf654cbee681328cf940e52097652c9dfc6 (diff) | |
download | cpython-0f7c25d20f68f5fc7f51e05161b0eeb34b8050de.zip cpython-0f7c25d20f68f5fc7f51e05161b0eeb34b8050de.tar.gz cpython-0f7c25d20f68f5fc7f51e05161b0eeb34b8050de.tar.bz2 |
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.
Diffstat (limited to 'Lib/urllib.py')
-rw-r--r-- | Lib/urllib.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Lib/urllib.py b/Lib/urllib.py index ad0e72b..c6751b8 100644 --- a/Lib/urllib.py +++ b/Lib/urllib.py @@ -342,7 +342,7 @@ class URLopener: '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 @@ class URLopener: '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: |