Jim Jewett
                                 added the comment:

The change still missed 
the httpS copy.  I'm 
attaching a minimal change.

I think it might be better 
to just combine the 
methods -- as was already 
done in Py3K.  
Unfortunately, the py3K 
code doesn't run cleanly 
in 2.5, and I haven't yet 
had a chance to test a 
backported equivalent.  
(Hopefully tonight.)

__________________________________
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1177>
__________________________________
*** urllibhead.py 
--- urllib.py 
***************
*** 435,441 ****
                  # something went wrong with the HTTP status line
                  raise IOError, ('http protocol error', 0,
                                  'got a bad status line', None)
!             if errcode == 200:
                  return addinfourl(fp, headers, "https:" + url)
              else:
                  if data is None:
--- 435,443 ----
                  # something went wrong with the HTTP status line
                  raise IOError, ('http protocol error', 0,
                                  '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):
                  return addinfourl(fp, headers, "https:" + url)
              else:
                  if data is None:
_______________________________________________
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to