merge headers in OperatorTrait::sendRequest

Author: k0ka

src/Common/Api/OperatorTrait.php

@@ -101,8 +101,13 @@ protected function sendRequest(Operation $operation, array $userValues = [], boo
 
         $uri = Utils::uri_template($operation->getPath(), $userValues);
 
-        if (array_key_exists('requestOptions', $userValues)) {
+        if (isset($userValues['requestOptions'])) {
             $options += $userValues['requestOptions'];
+
+            // headers are always created in options, merge them
+            if (isset($userValues['requestOptions']['headers'])) {
+                $options['headers'] = array_merge($options['headers'], $userValues['requestOptions']['headers']);
+            }
         }
 
         $options['openstack.skip_auth'] = $operation->getSkipAuth();

tests/unit/Common/Api/OperatorTraitTest.php

@@ -111,6 +111,30 @@ public function test_guzzle_options_are_forwarded()
             'requestOptions' => ['stream' => true],
         ]);
     }
+
+    public function test_it_sends_custom_headers_in_request_options()
+    {
+        $this->client
+            ->requestAsync('GET', 'test',
+                [
+                    'headers' => [
+                        'Access-Control-Allow-Origin'  => '*',
+                        'Access-Control-Allow-Methods' => 'GET, POST, OPTIONS',
+                    ],
+                    'openstack.skip_auth' => false,
+                ])
+            ->shouldBeCalled()
+            ->willReturn(new Promise());
+
+        $this->operator->executeAsync($this->def, [
+            'requestOptions' => [
+                'headers' => [
+                    'Access-Control-Allow-Origin'  => '*',
+                    'Access-Control-Allow-Methods' => 'GET, POST, OPTIONS',
+                ],
+            ],
+        ]);
+    }
 }