[HttpClient] fix collecting debug info on destruction of CurlResponse

Author: nicolas-grekas

Response/CurlResponse.php

@@ -225,13 +225,15 @@ public function getContent(bool $throw = true): string
 
     public function __destruct()
     {
-        curl_setopt($this->handle, \CURLOPT_VERBOSE, false);
+        try {
+            if (null === $this->timeout) {
+                return; // Unused pushed response
+            }
 
-        if (null === $this->timeout) {
-            return; // Unused pushed response
+            $this->doDestruct();
+        } finally {
+            curl_setopt($this->handle, \CURLOPT_VERBOSE, false);
         }
-
-        $this->doDestruct();
     }
 
     /**

Tests/HttpClientTestCase.php

@@ -167,4 +167,16 @@ public function testHandleIsRemovedOnException()
             $this->assertCount(0, $clientState->openHandles);
         }
     }
+
+    public function testDebugInfoOnDestruct()
+    {
+        $client = $this->getHttpClient(__FUNCTION__);
+
+        $traceInfo = [];
+        $client->request('GET', 'http://localhost:8057', ['on_progress' => function (int $dlNow, int $dlSize, array $info) use (&$traceInfo) {
+            $traceInfo = $info;
+        }]);
+
+        $this->assertNotEmpty($traceInfo['debug']);
+    }
 }