修复因 PHP cURL bug 导致的内存泄漏问题

src/HttpRequest.php

@@ -239,6 +239,7 @@ public function open()
      */
     public function close()
     {
+        $this->handler->close();
         $this->handler = null;
     }
 

src/YurunHttp/Handler/Curl.php

@@ -94,7 +94,12 @@ public function __construct()
         $this->initCookieManager();
     }
 
-    public function __destruct()
+    /**
+     * 关闭并释放所有资源
+     *
+     * @return void
+     */
+    public function close()
     {
         if($this->handler)
         {
@@ -131,8 +136,6 @@ public function send($request)
         $uri = $this->request->getUri();
         $isLocation = false;
         $statusCode = 0;
-        $lastMethod = null;
-        $copyHandler = curl_copy_handle($this->handler);
         $redirectCount = 0;
         do{
             // 请求方法
@@ -152,12 +155,7 @@ public function send($request)
             {
                 $bodyContent = false;
             }
-            if($lastMethod && 'GET' !== $lastMethod && 'GET' === $method)
-            {
-                $this->handler = curl_copy_handle($copyHandler);
-            }
             $this->buildCurlHandlerEx($this->request, $this->handler, $uri, $method, $bodyContent);
-            $lastMethod = $method;
             $retry = $this->request->getAttribute(Attributes::RETRY, 0);
             for($i = 0; $i <= $retry; ++$i)
             {

src/YurunHttp/Handler/IHandler.php

@@ -48,4 +48,11 @@ public function getHandler();
      */
     public function coBatch($requests, $timeout = null);
 
+    /**
+     * 关闭并释放所有资源
+     *
+     * @return void
+     */
+    public function close();
+
 }

src/YurunHttp/Handler/Swoole.php

@@ -57,7 +57,12 @@ public function __construct()
         $this->http2ConnectionManager = new Http2ConnectionManager;
     }
 
-    public function __destruct()
+    /**
+     * 关闭并释放所有资源
+     *
+     * @return void
+     */
+    public function close()
     {
         $this->httpConnectionManager->close();
         $this->http2ConnectionManager->close();

tests/unit/HttpRequestTest/HttpRequestTest.php

@@ -480,4 +480,19 @@ public function testCoBatch()
         });
     }
 
+    public function testMemoryLeak()
+    {
+        $this->call(function(){
+            $memorys = [1, 2, 3, 4, 5];
+            for($i = 0; $i < 5; ++$i)
+            {
+                $http = new HttpRequest;
+                $http->get($this->host);
+                $memorys[$i] = memory_get_usage();
+            }
+            unset($memorys[0]);
+            $this->assertEquals(1, count(array_unique($memorys)));
+        });
+    }
+
 }