➜
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2008-04-08 15:19 by djc, last changed 2022-04-11 14:56 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| urllib2_2585.diff | gpolo, 2008-04-09 15:09 | |||
| Messages (3) | |||
|---|---|---|---|
| msg65168 - (view) | Author: Dirkjan Ochtman (djc) *  |
Date: 2008-04-08 15:19 | |
djc@enrai tests $ python2.6 Python 2.6a2+ (trunk, Apr 4 2008, 20:21:45) [GCC 4.1.2 20070214 ( (gdc 0.24, using dmd 1.020)) (Gentoo 4.1.2 p1.0.2)] on linux2 Type "help", "copyright", "credits" or "license" for more information. >>> import urllib2 >>> try: ... urllib2.urlopen('http://example.com/weird') ... except urllib2.HTTPError, inst: ... print inst.code ... None >>> urllib.addinfourl.__init__() was changed in r60133 to set self.code. Unfortunately, this overrides HTTPError.code, which is probably not good. |
|||
| msg65239 - (view) | Author: Guilherme Polo (gpolo) * |
Date: 2008-04-09 15:09 | |
It seems the fix is just passing code to addinfourl __init__, not sure if someone forgot this or this is just totally wrong. Patch added. |
|||
| msg65248 - (view) | Author: Georg Brandl (georg.brandl) * |
Date: 2008-04-09 17:58 | |
Thanks Guilherme, that is indeed a detail that I forgot. Fixed in r62246. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:56:33 | admin | set | github: 46837 |
| 2008-04-09 17:58:06 | georg.brandl | set | status: open -> closed resolution: fixed messages: + msg65248 |
| 2008-04-09 15:09:49 | gpolo | set | files:
+ urllib2_2585.diff nosy: + gpolo messages: + msg65239 keywords: + patch |
| 2008-04-08 20:35:13 | djc | set | nosy: + georg.brandl, - birkenfeld |
| 2008-04-08 15:19:24 | djc | create | |