fix additional params evaluation to include string array params

Author: sbiscigl

src/aws-cpp-sdk-core/include/smithy/client/AwsSmithyClient.h

@@ -169,12 +169,15 @@ namespace client
                 const auto& epParams = ctx.m_pRequest->GetEndpointContextParams();
                 for (const auto& epParam : epParams) {
                     using ParameterType = Aws::Endpoint::EndpointParameter::ParameterType;
-                    if(epParam.GetStoredType() == ParameterType::STRING)
-                        identityParams.additionalProperties.insert({epParam.GetName(), epParam.GetStrValueNoCheck()});
-                    else if (epParam.GetStoredType() == ParameterType::BOOLEAN)
-                        identityParams.additionalProperties.insert({epParam.GetName(), epParam.GetBoolValueNoCheck()});
-                    else
-                        assert(!"Unknown endpoint parameter!");
+                    if(epParam.GetStoredType() == ParameterType::STRING) {
+                      identityParams.additionalProperties.insert({epParam.GetName(), epParam.GetStrValueNoCheck()});
+                    } else if (epParam.GetStoredType() == ParameterType::BOOLEAN) {
+                      identityParams.additionalProperties.insert({epParam.GetName(), epParam.GetBoolValueNoCheck()});
+                    } else if (epParam.GetStoredType() == ParameterType::STRING_ARRAY) {
+                      identityParams.additionalProperties.insert({epParam.GetName(), epParam.GetStrArrayValueNoCheck()});
+                    } else {
+                      assert(!"Unknown endpoint parameter!");
+                    }
                 }
                 const auto& serviceParams = ctx.m_pRequest->GetServiceSpecificParameters();
                 if (serviceParams) {

src/aws-cpp-sdk-core/include/smithy/identity/auth/AuthSchemeResolverBase.h

@@ -26,7 +26,8 @@ class DefaultAuthSchemeResolverParameters
   Aws::UnorderedMap<Aws::String, Aws::Crt::Variant<Aws::String,
           bool,
           Aws::Client::AWSAuthV4Signer::PayloadSigningPolicy,
-          Aws::Auth::AWSSigningAlgorithm > > additionalProperties;
+          Aws::Auth::AWSSigningAlgorithm,
+          Aws::Vector<Aws::String>>> additionalProperties;
 
 };
 

tests/aws-cpp-sdk-dynamodb-integration-tests/TableOperationTest.cpp

@@ -31,6 +31,7 @@
 #include <aws/dynamodb/model/ScanRequest.h>
 #include <aws/dynamodb/model/UpdateItemRequest.h>
 #include <aws/dynamodb/model/DeleteItemRequest.h>
+#include <aws/dynamodb/model/BatchGetItemRequest.h>
 #include <aws/testing/TestingEnvironment.h>
 #include <aws/core/utils/UUID.h>
 
@@ -1523,5 +1524,23 @@ TEST_F(TableOperationTest, TestEndpointOverride)
     }
 }
 
+TEST_F(TableOperationTest, TestBatchGetItem) {
+  Aws::String table_name = BuildTableName(BASE_SIMPLE_TABLE);
+  CreateTable(table_name, 10, 10);
+
+  AttributeValue value{"wheres everyone going, bingo?"};
+  const auto put_item_result = m_client->PutItem(PutItemRequest().WithTableName(table_name).WithItem({{HASH_KEY_NAME, value}}));
+  AWS_ASSERT_SUCCESS(put_item_result);
+
+  const auto batch_get_item_result = m_client->BatchGetItem(BatchGetItemRequest()
+    .WithRequestItems({
+      {
+        table_name, KeysAndAttributes().WithKeys({{{HASH_KEY_NAME, value}}})
+      }
+    }));
+  AWS_ASSERT_SUCCESS(batch_get_item_result);
+  EXPECT_EQ(1ul, batch_get_item_result.GetResult().GetResponses().size());
+}
+
 } // anonymous namespace