Revert "bug #45813 [HttpClient] Move Content-Type after Content-Length (nicolas-grekas)"

This reverts commit 13e0671ff9222d6797c5a44593c1a09f65e8a738, reversing
changes made to 01f674975a2dd27340aab9b6e7402bc0aee8423f.

Author: nicolas-grekas

HttpClientTrait.php

@@ -159,13 +159,6 @@ private static function prepareRequest(?string $method, ?string $url, array $opt
         }
 
         $options['max_duration'] = isset($options['max_duration']) ? (float) $options['max_duration'] : 0;
-
-        if (isset($options['normalized_headers']['content-length']) && $contentType = $options['normalized_headers']['content-type'] ?? null) {
-            // Move Content-Type after Content-Length, see https://bugs.php.net/44603
-            unset($options['normalized_headers']['content-type']);
-            $options['normalized_headers']['content-type'] = $contentType;
-        }
-
         $options['headers'] = array_merge(...array_values($options['normalized_headers']));
 
         return [$url, $options];

Tests/ScopingHttpClientTest.php

@@ -73,7 +73,7 @@ public function testMatchingUrlsAndOptions()
 
         $response = $client->request('GET', 'http://example.com/foo-bar', ['json' => ['url' => 'http://example.com']]);
         $requestOptions = $response->getRequestOptions();
-        $this->assertSame('Content-Type: application/json', $requestOptions['headers'][3]);
+        $this->assertSame('Content-Type: application/json', $requestOptions['headers'][1]);
         $requestJson = json_decode($requestOptions['body'], true);
         $this->assertSame('http://example.com', $requestJson['url']);
         $this->assertSame('X-FooBar: '.$defaultOptions['.*/foo-bar']['headers']['X-FooBar'], $requestOptions['headers'][0]);