[HttpClient] fix handling of 3xx with no Location header - ignore Content-Length when no body is expected

Author: nicolas-grekas

HttpClient/Test/Fixtures/web/index.php

@@ -82,6 +82,11 @@
         header('Location: ..', true, 302);
         break;
 
+    case '/304':
+        header('Content-Length: 10', true, 304);
+        echo '12345';
+        return;
+
     case '/307':
         header('Location: http://localhost:8057/post', true, 307);
         break;

HttpClient/Test/HttpClientTestCase.php

@@ -260,6 +260,17 @@ public function testBadRequestBody()
         $response->getStatusCode();
     }
 
+    public function test304()
+    {
+        $client = $this->getHttpClient(__FUNCTION__);
+        $response = $client->request('GET', 'http://localhost:8057/304', [
+            'headers' => ['If-Match' => '"abc"'],
+        ]);
+
+        $this->assertSame(304, $response->getStatusCode());
+        $this->assertSame('', $response->getContent(false));
+    }
+
     public function testRedirects()
     {
         $client = $this->getHttpClient(__FUNCTION__);