[zendframework/zendframework#4555] Do not double-decode gzipped respo…

…nses via cURL

- cURL's CURLOPT_ENCODING setting can allow auto-decoding a response.
  Detect if set appropriately, and, if so, strip the Content-Encoding
  header in the response.

src/Client/Adapter/Curl.php

@@ -427,6 +427,14 @@ public function write($method, $uri, $httpVersion = 1.1, $headers = array(), $bo
             $this->response = str_ireplace("Transfer-Encoding: chunked\r\n", '', $this->response);
         }
 
+        // cURL can automatically handle content encoding; prevent double-decoding from occurring
+        if (isset($this->config['curloptions'][CURLOPT_ENCODING])
+            && '' == $this->config['curloptions'][CURLOPT_ENCODING]
+            && stripos($this->response, "Content-Encoding: gzip\r\n")
+        ) {
+            $this->response = str_ireplace("Content-Encoding: gzip\r\n", '', $this->response);
+        }
+
         // Eliminate multiple HTTP responses.
         do {
             $parts  = preg_split('|(?:\r?\n){2}|m', $this->response, 2);

test/Client/CurlTest.php

@@ -347,4 +347,22 @@ public function testAuthorizeHeader()
 
         $this->assertContains($header, $curlInfo['request_header'], 'Expecting valid basic authorization header');
     }
+
+    /**
+     * @group 4555
+     */
+    public function testResponseDoesNotDoubleDecodeGzippedBody()
+    {
+        $this->client->setUri($this->baseuri . 'testCurlGzipData.php');
+        $adapter = new Adapter\Curl();
+        $adapter->setOptions(array(
+            'curloptions' => array(
+                CURLOPT_ENCODING => '',
+            ),
+        ));
+        $this->client->setAdapter($adapter);
+        $this->client->setMethod('GET');
+        $this->client->send();
+        $this->assertEquals('Success', $this->client->getResponse()->getBody());
+    }
 }

test/Client/_files/testCurlGzipData.php

@@ -0,0 +1,3 @@
+<?php
+header('Content-Encoding: gzip');
+echo gzcompress('Success');