Commit 08c08bb3 authored by Andrew M. Kuchling's avatar Andrew M. Kuchling

[Bug #912845] urllib2 only checks for a 200 return code, but 206 is also legal...

[Bug #912845] urllib2 only checks for a 200 return code, but 206 is also legal if a Range: header was supplied.
(Actually, should the first 'if' statement be modified to allow any 2xx status code?)
parent 31352c5a
......@@ -468,7 +468,7 @@ class HTTPErrorProcessor(BaseHandler):
def http_response(self, request, response):
code, msg, hdrs = response.code, response.msg, response.info()
if code != 200:
if code not in (200, 206):
response = self.parent.error(
'http', request, response, code, msg, hdrs)
......@@ -996,7 +996,7 @@ class AbstractHTTPHandler(BaseHandler):
except socket.error, err: # XXX what error?
raise URLError(err)
if r.status == 200:
if r.status in (200, 206):
# Pick apart the HTTPResponse object to get the addinfourl
# object initialized properly
resp = addinfourl(r.fp, r.msg, req.get_full_url())
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment