From f91149e4a194621b08346ce3f110712a174e271c Mon Sep 17 00:00:00 2001
From: Georg Brandl <georg@python.org>
Date: Thu, 12 Jul 2007 08:05:45 +0000
Subject: [PATCH] Patch #1752270, #1750931: complain if urllib2 add_handler
 called without handler.

---
 Lib/test/test_urllib2.py | 6 ++++++
 Lib/urllib2.py           | 4 ++++
 2 files changed, 10 insertions(+)

diff --git a/Lib/test/test_urllib2.py b/Lib/test/test_urllib2.py
index c9dcf5ef7be..90e17717337 100644
--- a/Lib/test/test_urllib2.py
+++ b/Lib/test/test_urllib2.py
@@ -381,6 +381,12 @@ class MockPasswordManager:
 
 class OpenerDirectorTests(unittest.TestCase):
 
+    def test_add_non_handler(self):
+        class NonHandler(object):
+            pass
+        self.assertRaises(TypeError,
+                          OpenerDirector().add_handler, NonHandler())
+
     def test_badly_named_methods(self):
         # test work-around for three methods that accidentally follow the
         # naming conventions for handler methods
diff --git a/Lib/urllib2.py b/Lib/urllib2.py
index 415c991ad32..b2cec727fa7 100644
--- a/Lib/urllib2.py
+++ b/Lib/urllib2.py
@@ -295,6 +295,10 @@ class OpenerDirector:
         self.process_request = {}
 
     def add_handler(self, handler):
+        if not hasattr(handler, "add_parent"):
+            raise TypeError("expected BaseHandler instance, got %r" %
+                            type(handler))
+
         added = False
         for meth in dir(handler):
             if meth in ["redirect_request", "do_open", "proxy_open"]: