diff options
author | Shawn O. Pearce <sop@google.com> | 2011-10-11 09:24:07 -0700 |
---|---|---|
committer | Shawn O. Pearce <sop@google.com> | 2011-10-11 09:24:07 -0700 |
commit | 29472463ba601e9c0513eadb19470e435b2601a1 (patch) | |
tree | 5dc815bb61bd53c60fd301f429d779994ef07f9b /main.py | |
parent | c325dc35f621fe24d0460bb14547cdb51e00c00b (diff) | |
download | git-repo-29472463ba601e9c0513eadb19470e435b2601a1.tar.gz |
Work around Python 2.7 failure to initialize base class
urllib2 returns a malformed HTTPError object in certain situations.
For example, urllib2 has a couple of places where it creates an
HTTPError object with no fp:
if self.retried > 5:
# retry sending the username:password 5 times before failing.
raise HTTPError(req.get_full_url(), 401, "basic auth failed",
headers, None)
When it does that, HTTPError's ctor doesn't call through to
addinfourl's ctor:
# The addinfourl classes depend on fp being a valid file
# object. In some cases, the HTTPError may not have a valid
# file object. If this happens, the simplest workaround is to
# not initialize the base classes.
if fp is not None:
self.__super_init(fp, hdrs, url, code)
Which means the 'headers' slot in addinfourl is not initialized and
info() fails. It is completely insane that urllib2 decides not to
initialize its own base class sometimes.
Change-Id: I32a0d738f71bdd7d38d86078b71d9001e26f1ec3
Signed-off-by: Shawn O. Pearce <sop@google.com>
Diffstat (limited to 'main.py')
0 files changed, 0 insertions, 0 deletions