fix: allow configuration of php-http/socket-client >=2.0

Author: qkdreyer

src/Client.php

@@ -32,7 +32,9 @@ class Client implements HttpClient
     /**
      * Constructor.
      *
-     * @param array{remote_socket?: string|null, timeout?: int, stream_context?: resource, stream_context_options?: array<string, mixed>, stream_context_param?: array<string, mixed>, ssl?: ?boolean, write_buffer_size?: int, ssl_method?: int} $config
+     * @param array{remote_socket?: string|null, timeout?: int, stream_context?: resource, stream_context_options?: array<string, mixed>, stream_context_param?: array<string, mixed>, ssl?: ?boolean, write_buffer_size?: int, ssl_method?: int} $configOrResponseFactory
+     * @param array|null                                                                                                                                                                                                                          $config2                 Mistake when refactoring the constructor from version 1 to version 2 - used as $config if set and $configOrResponseFactory is a response factory instance
+     * @param array                                                                                                                                                                                                                               $config                  intended for version 1 BC, used as $config if $config2 is not set and $configOrResponseFactory is a response factory instance
      *
      * string|null          remote_socket          Remote entrypoint (can be a tcp or unix domain address)
      * int                  timeout                Timeout before canceling request
@@ -43,17 +45,17 @@ class Client implements HttpClient
      * int                  write_buffer_size      Buffer when writing the request body, defaults to 8192
      * int                  ssl_method             Crypto method for ssl/tls, see PHP doc, defaults to STREAM_CRYPTO_METHOD_TLSv1_2_CLIENT
      */
-    public function __construct($config1 = [], $config2 = null, array $config = [])
+    public function __construct($configOrResponseFactory = [], $config2 = null, array $config = [])
     {
-        if (\is_array($config1)) {
-            $this->config = $this->configure($config1);
+        if (\is_array($configOrResponseFactory)) {
+            $this->config = $this->configure($configOrResponseFactory);
 
             return;
         }
 
-        @trigger_error('Passing a Psr\Http\Message\ResponseFactoryInterface and a Psr\Http\Message\StreamFactoryInterface to SocketClient is deprecated, and will be removed in 3.0, you should only pass config options.', E_USER_DEPRECATED);
+        @trigger_error('Passing a Psr\Http\Message\ResponseFactoryInterface to SocketClient is deprecated, and will be removed in 3.0, you should only pass config options.', E_USER_DEPRECATED);
 
-        $this->config = $this->configure($config);
+        $this->config = $this->configure($config2 ?: $config);
     }
 
     /**