From 8d904c39819eff6950bb4e712b28c434f0131af4 Mon Sep 17 00:00:00 2001 From: Amaury Forgeot d'Arc Date: Tue, 22 Apr 2008 21:17:18 +0000 Subject: Issue #2670: urllib2.build_opener() failed when two handlers derive the same default base class. Backport of r62463. --- Lib/test/test_urllib2.py | 6 ++++++ Lib/urllib2.py | 6 +++--- Misc/NEWS | 3 +++ 3 files changed, 12 insertions(+), 3 deletions(-) diff --git a/Lib/test/test_urllib2.py b/Lib/test/test_urllib2.py index 9934d37..96a7db9 100644 --- a/Lib/test/test_urllib2.py +++ b/Lib/test/test_urllib2.py @@ -1038,6 +1038,12 @@ class MiscTests(unittest.TestCase): o = build_opener(urllib2.HTTPHandler()) self.opener_has_handler(o, urllib2.HTTPHandler) + # Issue2670: multiple handlers sharing the same base class + class MyOtherHTTPHandler(urllib2.HTTPHandler): pass + o = build_opener(MyHTTPHandler, MyOtherHTTPHandler) + self.opener_has_handler(o, MyHTTPHandler) + self.opener_has_handler(o, MyOtherHTTPHandler) + def opener_has_handler(self, opener, handler_class): for h in opener.handlers: if h.__class__ == handler_class: diff --git a/Lib/urllib2.py b/Lib/urllib2.py index 3578e7a..50d7aaf 100644 --- a/Lib/urllib2.py +++ b/Lib/urllib2.py @@ -447,14 +447,14 @@ def build_opener(*handlers): FTPHandler, FileHandler, HTTPErrorProcessor] if hasattr(httplib, 'HTTPS'): default_classes.append(HTTPSHandler) - skip = [] + skip = set() for klass in default_classes: for check in handlers: if isclass(check): if issubclass(check, klass): - skip.append(klass) + skip.add(klass) elif isinstance(check, klass): - skip.append(klass) + skip.add(klass) for klass in skip: default_classes.remove(klass) diff --git a/Misc/NEWS b/Misc/NEWS index b9ce278..d66e26c 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -45,6 +45,9 @@ Core and builtins Library ------- +- Issue #2670: Fix a failure in urllib2.build_opener(), when passed two + handlers that derive the same default base class. + - Issue #2495: tokenize.untokenize now inserts a space between two consecutive string literals; previously, ["" ""] was rendered as [""""], which is incorrect python code. -- cgit v0.12