Raise exception for shutdown on a connection already released to the pool (#3601)

alexprabhat99 · pquentin · web-flow · commit 8e32e60d9024 · 2025-06-15T10:45:25.000+04:00
Co-authored-by: Quentin Pradet &lt;quentin.pradet@gmail.com&gt;
diff --git a/changelog/3581.bugfix.rst b/changelog/3581.bugfix.rst
@@ -0,0 +1 @@
+Raise exception for HTTPResponse.shutdown on a connection already released to the pool.
diff --git a/src/urllib3/response.py b/src/urllib3/response.py
@@ -1100,6 +1100,10 @@ def readable(self) -> bool:
     def shutdown(self) -> None:
         if not self._sock_shutdown:
             raise ValueError("Cannot shutdown socket as self._sock_shutdown is not set")
+        if self._connection is None:
+            raise RuntimeError(
+                "Cannot shutdown as connection has already been released to the pool"
+            )
         self._sock_shutdown(socket.SHUT_RD)
 
     def close(self) -> None:
diff --git a/test/with_dummyserver/test_connectionpool.py b/test/with_dummyserver/test_connectionpool.py
@@ -912,6 +912,18 @@ def test_cleanup_on_connection_error(self) -> None:
             # the pool should still contain poolsize elements
             assert http.pool.qsize() == http.pool.maxsize
 
+    def test_shutdown_on_connection_released_to_pool(self) -> None:
+        with HTTPConnectionPool(self.host, self.port) as pool:
+            resp = pool.urlopen("GET", "/", preload_content=False)
+            resp.drain_conn()
+            resp.release_conn()
+
+        with pytest.raises(
+            RuntimeError,
+            match="Cannot shutdown as connection has already been released to the pool",
+        ):
+            resp.shutdown()
+
     def test_mixed_case_hostname(self) -> None:
         with HTTPConnectionPool("LoCaLhOsT", self.port) as pool:
             response = pool.request("GET", f"http://LoCaLhOsT:{self.port}/")

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+Raise exception for HTTPResponse.shutdown on a connection already released to the pool.`