Add an accept header in JSON request to help 3rd party services like …

…localstack
async-aws · Jun 3, 2024 · 37c60a6 · 37c60a6
1 parent f840c9c
commit 37c60a6
Show file tree

Hide file tree

Showing 269 changed files with 563 additions and 78 deletions.
diff --git a/src/CodeGenerator/src/Generator/InputGenerator.php b/src/CodeGenerator/src/Generator/InputGenerator.php
@@ -389,9 +389,9 @@ private function inputClassRequestGetters(StructureShape $inputShape, ClassBuild
         $serializer = $this->serializer->get($operation->getService());
 
         if ((null !== $payloadProperty = $inputShape->getPayload()) && $inputShape->getMember($payloadProperty)->isStreaming()) {
-            $body['header'] = '$headers = [];' . "\n";
+            $body['header'] = '$headers = ' . $serializer->getHeaders($operation, false) . ';' . "\n";
         } else {
-            $body['header'] = '$headers = ' . $serializer->getHeaders($operation) . ';' . "\n";
+            $body['header'] = '$headers = ' . $serializer->getHeaders($operation, true) . ';' . "\n";
         }
 
         $body['querystring'] = '$query = [];' . "\n";

diff --git a/src/CodeGenerator/src/Generator/RequestSerializer/JsonRpcSerializer.php b/src/CodeGenerator/src/Generator/RequestSerializer/JsonRpcSerializer.php
@@ -15,18 +15,19 @@
  */
 class JsonRpcSerializer extends RestJsonSerializer
 {
-    public function getHeaders(Operation $operation): string
+    public function getHeaders(Operation $operation, bool $withPayload): string
     {
-        return strtr(<<<PHP
-[
-    'Content-Type' => 'application/x-amz-json-VERSION',
-    'X-Amz-Target' => 'TARGET',
-];
+        if (!$withPayload) {
+            return "['Accept' => 'application/json']";
+        }
 
-PHP
-            , [
-                'VERSION' => number_format($operation->getService()->getJsonVersion(), 1),
-                'TARGET' => sprintf('%s.%s', $operation->getService()->getTargetPrefix(), $operation->getName()),
-            ]);
+        return strtr("[
+            'Content-Type' => 'application/x-amz-json-VERSION',
+            'X-Amz-Target' => 'TARGET',
+            'Accept' => 'application/json',
+        ]", [
+            'VERSION' => number_format($operation->getService()->getJsonVersion(), 1),
+            'TARGET' => sprintf('%s.%s', $operation->getService()->getTargetPrefix(), $operation->getName()),
+        ]);
     }
 }
diff --git a/src/CodeGenerator/src/Generator/RequestSerializer/QuerySerializer.php b/src/CodeGenerator/src/Generator/RequestSerializer/QuerySerializer.php
@@ -43,9 +43,13 @@ public function __construct(NamespaceRegistry $namespaceRegistry, RequirementsRe
         $this->requirementsRegistry = $requirementsRegistry;
     }
 
-    public function getHeaders(Operation $operation): string
+    public function getHeaders(Operation $operation, bool $withPayload): string
     {
-        return '["content-type" => "application/x-www-form-urlencoded"]';
+        if (!$withPayload) {
+            return '[]';
+        }
+
+        return "['content-type' => 'application/x-www-form-urlencoded']";
     }
 
     public function generateRequestBody(Operation $operation, StructureShape $shape): SerializerResultBody

diff --git a/src/CodeGenerator/src/Generator/RequestSerializer/RestJsonSerializer.php b/src/CodeGenerator/src/Generator/RequestSerializer/RestJsonSerializer.php
@@ -43,9 +43,16 @@ public function __construct(NamespaceRegistry $namespaceRegistry, RequirementsRe
         $this->requirementsRegistry = $requirementsRegistry;
     }
 
-    public function getHeaders(Operation $operation): string
+    public function getHeaders(Operation $operation, bool $withPayload): string
     {
-        return '["content-type" => "application/json"]';
+        if (!$withPayload) {
+            return "['Accept' => 'application/json']";
+        }
+
+        return "[
+            'Content-Type' => 'application/json',
+            'Accept' => 'application/json',
+        ]";
     }
 
     public function generateRequestBody(Operation $operation, StructureShape $shape): SerializerResultBody

diff --git a/src/CodeGenerator/src/Generator/RequestSerializer/RestXmlSerializer.php b/src/CodeGenerator/src/Generator/RequestSerializer/RestXmlSerializer.php
@@ -40,9 +40,13 @@ public function __construct(NamespaceRegistry $namespaceRegistry, RequirementsRe
         $this->requirementsRegistry = $requirementsRegistry;
     }
 
-    public function getHeaders(Operation $operation): string
+    public function getHeaders(Operation $operation, bool $withPayload): string
     {
-        return '["content-type" => "application/xml"]';
+        if (!$withPayload) {
+            return '[]';
+        }
+
+        return "['content-type' => 'application/xml']";
     }
 
     public function generateRequestBody(Operation $operation, StructureShape $shape): SerializerResultBody

diff --git a/src/CodeGenerator/src/Generator/RequestSerializer/Serializer.php b/src/CodeGenerator/src/Generator/RequestSerializer/Serializer.php
@@ -27,5 +27,5 @@ public function generateRequestBody(Operation $operation, StructureShape $shape)
      */
     public function generateRequestBuilder(StructureShape $shape, bool $needsChecks): SerializerResultBuilder;
 
-    public function getHeaders(Operation $operation): string;
+    public function getHeaders(Operation $operation, bool $withPayload): string;
 }
diff --git a/src/Service/AppSync/CHANGELOG.md b/src/Service/AppSync/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## NOT RELEASED
 
+### Changed
+
+- Add `Accept: application/json` header in request to fix incompatibility with 3rd party providers
+
 ## 2.1.0
 
 ### Added

diff --git a/src/Service/AppSync/src/Input/CreateResolverRequest.php b/src/Service/AppSync/src/Input/CreateResolverRequest.php
@@ -278,7 +278,10 @@ public function getTypeName(): ?string
     public function request(): Request
     {
         // Prepare headers
-        $headers = ['content-type' => 'application/json'];
+        $headers = [
+            'Content-Type' => 'application/json',
+            'Accept' => 'application/json',
+        ];
 
         // Prepare query
         $query = [];

diff --git a/src/Service/AppSync/src/Input/DeleteResolverRequest.php b/src/Service/AppSync/src/Input/DeleteResolverRequest.php
@@ -86,7 +86,10 @@ public function getTypeName(): ?string
     public function request(): Request
     {
         // Prepare headers
-        $headers = ['content-type' => 'application/json'];
+        $headers = [
+            'Content-Type' => 'application/json',
+            'Accept' => 'application/json',
+        ];
 
         // Prepare query
         $query = [];

diff --git a/src/Service/AppSync/src/Input/GetSchemaCreationStatusRequest.php b/src/Service/AppSync/src/Input/GetSchemaCreationStatusRequest.php
@@ -52,7 +52,10 @@ public function getApiId(): ?string
     public function request(): Request
     {
         // Prepare headers
-        $headers = ['content-type' => 'application/json'];
+        $headers = [
+            'Content-Type' => 'application/json',
+            'Accept' => 'application/json',
+        ];
 
         // Prepare query
         $query = [];

diff --git a/src/Service/AppSync/src/Input/ListApiKeysRequest.php b/src/Service/AppSync/src/Input/ListApiKeysRequest.php
@@ -83,7 +83,10 @@ public function getNextToken(): ?string
     public function request(): Request
     {
         // Prepare headers
-        $headers = ['content-type' => 'application/json'];
+        $headers = [
+            'Content-Type' => 'application/json',
+            'Accept' => 'application/json',
+        ];
 
         // Prepare query
         $query = [];

diff --git a/src/Service/AppSync/src/Input/ListResolversRequest.php b/src/Service/AppSync/src/Input/ListResolversRequest.php
@@ -100,7 +100,10 @@ public function getTypeName(): ?string
     public function request(): Request
     {
         // Prepare headers
-        $headers = ['content-type' => 'application/json'];
+        $headers = [
+            'Content-Type' => 'application/json',
+            'Accept' => 'application/json',
+        ];
 
         // Prepare query
         $query = [];

diff --git a/src/Service/AppSync/src/Input/StartSchemaCreationRequest.php b/src/Service/AppSync/src/Input/StartSchemaCreationRequest.php
@@ -69,7 +69,10 @@ public function getDefinition(): ?string
     public function request(): Request
     {
         // Prepare headers
-        $headers = ['content-type' => 'application/json'];
+        $headers = [
+            'Content-Type' => 'application/json',
+            'Accept' => 'application/json',
+        ];
 
         // Prepare query
         $query = [];

diff --git a/src/Service/AppSync/src/Input/UpdateApiKeyRequest.php b/src/Service/AppSync/src/Input/UpdateApiKeyRequest.php
@@ -100,7 +100,10 @@ public function getId(): ?string
     public function request(): Request
     {
         // Prepare headers
-        $headers = ['content-type' => 'application/json'];
+        $headers = [
+            'Content-Type' => 'application/json',
+            'Accept' => 'application/json',
+        ];
 
         // Prepare query
         $query = [];

diff --git a/src/Service/AppSync/src/Input/UpdateDataSourceRequest.php b/src/Service/AppSync/src/Input/UpdateDataSourceRequest.php
@@ -259,7 +259,10 @@ public function getType(): ?string
     public function request(): Request
     {
         // Prepare headers
-        $headers = ['content-type' => 'application/json'];
+        $headers = [
+            'Content-Type' => 'application/json',
+            'Accept' => 'application/json',
+        ];
 
         // Prepare query
         $query = [];

diff --git a/src/Service/AppSync/src/Input/UpdateResolverRequest.php b/src/Service/AppSync/src/Input/UpdateResolverRequest.php
@@ -278,7 +278,10 @@ public function getTypeName(): ?string
     public function request(): Request
     {
         // Prepare headers
-        $headers = ['content-type' => 'application/json'];
+        $headers = [
+            'Content-Type' => 'application/json',
+            'Accept' => 'application/json',
+        ];
 
         // Prepare query
         $query = [];

diff --git a/src/Service/Athena/CHANGELOG.md b/src/Service/Athena/CHANGELOG.md
@@ -4,6 +4,7 @@
 
 ### Changed
 
+- Add `Accept: application/json` header in request to fix incompatibility with 3rd party providers
 - AWS enhancement: Documentation updates.
 
 ## 2.2.0

diff --git a/src/Service/Athena/src/Input/GetCalculationExecutionRequest.php b/src/Service/Athena/src/Input/GetCalculationExecutionRequest.php
@@ -55,6 +55,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.GetCalculationExecution',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/GetCalculationExecutionStatusRequest.php b/src/Service/Athena/src/Input/GetCalculationExecutionStatusRequest.php
@@ -55,6 +55,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.GetCalculationExecutionStatus',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/GetDataCatalogInput.php b/src/Service/Athena/src/Input/GetDataCatalogInput.php
@@ -70,6 +70,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.GetDataCatalog',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/GetDatabaseInput.php b/src/Service/Athena/src/Input/GetDatabaseInput.php
@@ -88,6 +88,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.GetDatabase',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/GetNamedQueryInput.php b/src/Service/Athena/src/Input/GetNamedQueryInput.php
@@ -55,6 +55,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.GetNamedQuery',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/GetQueryExecutionInput.php b/src/Service/Athena/src/Input/GetQueryExecutionInput.php
@@ -55,6 +55,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.GetQueryExecution',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/GetQueryResultsInput.php b/src/Service/Athena/src/Input/GetQueryResultsInput.php
@@ -87,6 +87,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.GetQueryResults',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/GetSessionRequest.php b/src/Service/Athena/src/Input/GetSessionRequest.php
@@ -55,6 +55,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.GetSession',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/GetSessionStatusRequest.php b/src/Service/Athena/src/Input/GetSessionStatusRequest.php
@@ -55,6 +55,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.GetSessionStatus',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/GetTableMetadataInput.php b/src/Service/Athena/src/Input/GetTableMetadataInput.php
@@ -105,6 +105,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.GetTableMetadata',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/GetWorkGroupInput.php b/src/Service/Athena/src/Input/GetWorkGroupInput.php
@@ -55,6 +55,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.GetWorkGroup',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/ListDatabasesInput.php b/src/Service/Athena/src/Input/ListDatabasesInput.php
@@ -103,6 +103,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.ListDatabases',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/ListNamedQueriesInput.php b/src/Service/Athena/src/Input/ListNamedQueriesInput.php
@@ -85,6 +85,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.ListNamedQueries',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/ListQueryExecutionsInput.php b/src/Service/Athena/src/Input/ListQueryExecutionsInput.php
@@ -85,6 +85,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.ListQueryExecutions',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/ListTableMetadataInput.php b/src/Service/Athena/src/Input/ListTableMetadataInput.php
@@ -134,6 +134,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.ListTableMetadata',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/StartCalculationExecutionRequest.php b/src/Service/Athena/src/Input/StartCalculationExecutionRequest.php
@@ -128,6 +128,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.StartCalculationExecution',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/StartQueryExecutionInput.php b/src/Service/Athena/src/Input/StartQueryExecutionInput.php
@@ -163,6 +163,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.StartQueryExecution',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/StartSessionRequest.php b/src/Service/Athena/src/Input/StartSessionRequest.php
@@ -142,6 +142,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.StartSession',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/StopCalculationExecutionRequest.php b/src/Service/Athena/src/Input/StopCalculationExecutionRequest.php
@@ -55,6 +55,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.StopCalculationExecution',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/StopQueryExecutionInput.php b/src/Service/Athena/src/Input/StopQueryExecutionInput.php
@@ -54,6 +54,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.StopQueryExecution',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query

diff --git a/src/Service/Athena/src/Input/TerminateSessionRequest.php b/src/Service/Athena/src/Input/TerminateSessionRequest.php
@@ -55,6 +55,7 @@ public function request(): Request
         $headers = [
             'Content-Type' => 'application/x-amz-json-1.1',
             'X-Amz-Target' => 'AmazonAthena.TerminateSession',
+            'Accept' => 'application/json',
         ];
 
         // Prepare query