[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Changes by Berker Peksag : -- stage: needs patch -> resolved ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: ht

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Roundup Robot added the comment: New changeset e1ba514ddcd2 by Senthil Kumaran in branch '3.2': Fix issue13211 - Document the reason attribute for urllib.error.HTTPError http://hg.python.org/cpython/rev/e1ba514ddcd2 -- ___ Python tracker

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Changes by Terry J. Reedy : -- versions: +Python 3.4 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mai

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Jason R. Coombs added the comment: After yet another commit, the build bots are green again: http://hg.python.org/cpython/rev/8fa1dc66de5d -- ___ Python tracker ___ ___

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Roundup Robot added the comment: New changeset a3ddee916808 by Jason R. Coombs in branch 'default': Pass positional arguments - HTTPError is not accepting keyword arguments. Reference #13211 and #12555. http://hg.python.org/cpython/rev/a3ddee916808 -- _

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Jason R. Coombs added the comment: Antoine, I think Arfrever is on to something here. It does appear the new test added to test_urllib2 is failing, but for reasons I don't exactly understand. I'm initializing the HTTPError instance according to its signature in urllib.error, but it fails to a

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Arfrever Frehtes Taifersar Arahesis added the comment: I suspect that this problem is caused by the fix for issue #12555. -- nosy: +Arfrever ___ Python tracker ___ _

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Antoine Pitrou added the comment: The 3.x buildbots are all broken: http://www.python.org/dev/buildbot/all/waterfall?category=3.x.stable&category=3.x.unstable -- nosy: +pitrou ___ Python tracker _

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Changes by Jason R. Coombs : -- resolution: -> fixed status: open -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Roundup Robot added the comment: New changeset ee94b89f65ab by Jason R. Coombs in branch '2.7': Issue #13211: Add .reason attribute to HTTPError to implement parent class (URLError) interface. http://hg.python.org/cpython/rev/ee94b89f65ab New changeset abfe76a19f63 by Jason R. Coombs in branch

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Changes by Jason R. Coombs : -- components: +Library (Lib) ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Jason R. Coombs added the comment: That was my point. If HTTPError is a subclass of URLError, then an HTTPError _is an_ URLError, and thus should implement the same public interface. The problem is better illustrated with this request: try: urllib.request.urlopen('http://api.wordnik.com/v4/

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Petri Lehtinen added the comment: My impression was that because HTTPError is a subclass of URLError, it should have the same attributes, and maybe some extra. A user reading the docs doesn't know whether it calls the base class __init__ or not. -- _

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Senthil Kumaran added the comment: Hi Jason & Petri, urllib2.HTTPError never had a reason attribute. In the docs, it is mentioned that only URLError has the reason attribute. The HTTPError sublasses URLError and addinforurl class, but the further initialization happens only in the addinforur

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Petri Lehtinen added the comment: Ok. Sounds that you've already evaluated this alternative, so I'm fine with it. -- ___ Python tracker ___ _

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Jason R. Coombs added the comment: My initial instinct was to agree - the status code is useful. However, in looking at the FTP code, it sometimes just sets other objects (socket.error for example) as the 'reason'. The docs say 'reason' is a string or another exception. I'm tempted to leave

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Petri Lehtinen added the comment: Perhaps the reason should include the status code, too? It makes HTTP errors much more useful, as you'll immediately see what's going on from the status code. -- ___ Python tracker

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Jason R. Coombs added the comment: I've created three changesets, addressing the issue in 2.7, 3.2, and 3.3, including tests. Please review and comment. If there are no objections, I'll push the changesets after 24 hours. -- ___ Python tracker

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Changes by Jason R. Coombs : Added file: http://bugs.python.org/file23627/fffeff7721c0.diff ___ Python tracker ___ ___ Python-bugs-list mailin

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Changes by Jason R. Coombs : -- hgrepos: +88 keywords: +needs review, patch ___ Python tracker ___ ___ Python-bugs-list mailing list U

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Changes by Senthil Kumaran : -- nosy: +orsenthil ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Jason R. Coombs added the comment: I scanned through the libs for Python 2.7, 3.2, and 3.3 and there is no construction of HTTPError that does not pass a string for msg. I believe it would be reasonable to alias reason to msg. I'll put together the changesets. --

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

Petri Lehtinen added the comment: > It appears the Python 3.2 docs no longer include documentation for URLError Both URLError and HTTPError are documented in 3.2 and 3.3: http://docs.python.org/py3k/library/urllib.error.html#urllib.error.URLError http://docs.python.org/dev/library/urllib.erro

[issue13211] urllib2.HTTPError does not have 'reason' attribute.

New submission from Jason R. Coombs : The urllib2 docs indicate that HTTPError is a subclass of URLError and that URLError has an attribute of 'reason', but HTTPError does not have this attribute. The docs should be updated to reflect this deviance. It appears the Python 3.2 docs no longer inc