CurlService should set its headers first so user's extraOptions can o…

…verride
instaclick · Dec 8, 2023 · 3314cd8 · 3314cd8
1 parent 1c4f303
commit 3314cd8
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/WebDriver/Service/CurlService.php b/lib/WebDriver/Service/CurlService.php
@@ -54,8 +54,8 @@ public function execute($requestMethod, $url, $parameters = null, $extraOptions
 
             case 'POST':
             case 'PUT':
-                $parameters =  ! $parameters || ! is_array($parameters)
-                    ? '{}'
+                $parameters = ! $parameters || ! is_array($parameters)
+                    ? '{}' // instead of json_encode(new \stdclass))
                     : json_encode($parameters);
 
                 curl_setopt($curl, CURLOPT_POSTFIELDS, $parameters);
@@ -74,12 +74,12 @@ public function execute($requestMethod, $url, $parameters = null, $extraOptions
                 break;
         }
 
+        curl_setopt($curl, CURLOPT_HTTPHEADER, $customHeaders);
+
         foreach (array_replace($this->defaultOptions, $extraOptions) as $option => $value) {
             curl_setopt($curl, $option, $value);
         }
 
-        curl_setopt($curl, CURLOPT_HTTPHEADER, $customHeaders);
-
         $rawResult = curl_exec($curl);
         $rawResult = is_string($rawResult) ? trim($rawResult) : '';