Only set Content-Type for POST requests

Author: bbatsche

src/Resolver/Service.php

@@ -49,18 +49,13 @@ public function resolve($definition)
             throw new \InvalidArgumentException('$definition must be an instance of ' . Definition::class);
         }
 
-        $headers = ['Content-type' => 'application/json'];
-
         $url             = $this->getIterator()->get('url', $definition);
         $query           = $this->getIterator()->get('query', $definition);
         $method          = $definition->has('method') ? $this->getIterator()->get('method', $definition) : 'POST';
         $variables       = $definition->has('variables') ? $this->getIterator()->get('variables', $definition) : [];
         $ignoreSSLErrors = $definition->has('ignoreSSLErrors')
             ? $this->getIterator()->get('ignoreSSLErrors', $definition)
             : false;
-        $headers = $definition->has('headers')
-            ? array_merge($headers, $this->getIterator()->get('headers', $definition))
-            : $headers;
         $requestParams = [
             'query'     => $query,
             'variables' => $variables,
@@ -73,14 +68,23 @@ public function resolve($definition)
                 CURLOPT_SSL_VERIFYPEER => !$ignoreSSLErrors,
             ],
         ]);
+
         $client->setMethod($method);
-        $client->setHeaders($headers);
+
         if ($method === 'POST') {
+            $headers = $definition->has('headers')
+                ? array_merge(['Content-type' => 'application/json'], $this->getIterator()->get('headers', $definition))
+                : ['Content-type' => 'application/json'];
+
             $client->setRawBody(json_encode($requestParams));
         } elseif ($method === 'GET') {
+            $headers = $definition->has('headers') ? $this->getIterator()->get('headers', $definition) : [];
+
             $client->setParameterGet($requestParams);
         }
 
+        $client->setHeaders($headers);
+
         $response = $client->send();
 
         return json_decode($response->getBody(), true);

test/Resolver/ServiceTest.php

@@ -133,7 +133,7 @@ public function testResolveWithConfiguration(): void
         $zendClientMock = Mockery::mock('overload:' . Client::class);
         $zendClientMock->shouldReceive('setMethod')->with('GET');
         $zendClientMock->shouldReceive('setHeaders')
-            ->with(['Content-type' => 'application/json', 'header' => 'headerValue']);
+            ->with(['header' => 'headerValue']);
         $zendClientMock->shouldReceive('setParameterGet')->with($expectedRequestBody);
         $zendClientMock->shouldReceive('send')->andReturn($responseMock);