From: Victor Stinner <victor.stinner@haypocalc.com>
Date: Fri, 17 Jun 2011 12:06:27 +0000 (+0200)
Subject: Issue #12133: AbstractHTTPHandler.do_open() of urllib.request closes the HTTP
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c74a6ba2d6c1f331896cf8dacc698b0b88c7e670;p=python

Issue #12133: AbstractHTTPHandler.do_open() of urllib.request closes the HTTP
connection if its getresponse() method fails with a socket error. Patch written
by Ezio Melotti.
---

diff --git a/Lib/test/test_urllib2.py b/Lib/test/test_urllib2.py
index e96f948a9b..e889bc3d10 100644
--- a/Lib/test/test_urllib2.py
+++ b/Lib/test/test_urllib2.py
@@ -293,6 +293,7 @@ class MockHTTPClass:
             self._tunnel_headers = headers
         else:
             self._tunnel_headers.clear()
+
     def request(self, method, url, body=None, headers=None):
         self.method = method
         self.selector = url
@@ -304,9 +305,13 @@ class MockHTTPClass:
         if self.raise_on_endheaders:
             import socket
             raise socket.error()
+
     def getresponse(self):
         return MockHTTPResponse(MockFile(), {}, 200, "OK")
 
+    def close(self):
+        pass
+
 class MockHandler:
     # useful for testing handler machinery
     # see add_ordered_mock_handlers() docstring
diff --git a/Lib/urllib2.py b/Lib/urllib2.py
index abc84b1dcb..26416193f6 100644
--- a/Lib/urllib2.py
+++ b/Lib/urllib2.py
@@ -1172,6 +1172,8 @@ class AbstractHTTPHandler(BaseHandler):
                 r = h.getresponse()
         except socket.error, err: # XXX what error?
             raise URLError(err)
+        finally:
+            h.close()
 
         # Pick apart the HTTPResponse object to get the addinfourl
         # object initialized properly.
diff --git a/Misc/NEWS b/Misc/NEWS
index bbd076a5e3..5fa417856a 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -16,6 +16,10 @@ Core and Builtins
 Library
 -------
 
+- Issue #12133: AbstractHTTPHandler.do_open() of urllib.request closes the HTTP
+  connection if its getresponse() method fails with a socket error. Patch
+  written by Ezio Melotti.
+
 - Issue #9284: Allow inspect.findsource() to find the source of doctest
   functions.