merge changes

.clang-tidy

@@ -5,9 +5,9 @@ HeaderFilterRegex: ''
 FormatStyle: 'none'
 CheckOptions:
   - key:             modernize-pass-by-value.ValuesOnly
-    value:           '1',
+    value:           '1'
   - key:             readability-implicit-bool-conversion.AllowPointerConditions
-    value:           '1',
+    value:           '1'
   - key:             readability-implicit-bool-conversion.AllowIntegerConditions
     value:           '1'
 

.cspell.json

@@ -57,7 +57,7 @@
     "MMSYSERR",
     "WAVEFORMATEX",
     "Unprepare",
-    "DDISABLE_IMDSV1",
+    "DDISABLE",
     "SENDREQUEST",
     "threadpool",
     "stdlib",

.github/workflows/clang-format.yml

@@ -29,23 +29,26 @@ jobs:
       # get names of the base and pr branches
       - name: Get the base and head branches
         id: fetchstep
+        env:
+          BASE_BRANCH: "${{ github.event.pull_request.base.ref }}"
+          HEAD_BRANCH: "${{ github.event.pull_request.head.ref }}"
         run: |
-          echo "BASE_BRANCH=${{ github.event.pull_request.base.ref }}" >> $GITHUB_ENV
-          echo "HEAD_BRANCH=${{ github.event.pull_request.head.ref }}" >> $GITHUB_ENV
           echo "Base branch: $BASE_BRANCH"
           echo "Head branch: $HEAD_BRANCH"
-          git fetch origin ${{ env.BASE_BRANCH }}
-          git fetch origin ${{ env.HEAD_BRANCH }}
+          git fetch origin "$BASE_BRANCH"
+          git fetch origin "$HEAD_BRANCH"
 
       #get differences on the PR branch excluding generated folder in the root
       - name: git diff
         id: diffstep
         env:
           ACTIONS_RUNNER_DEBUG: true
+          BASE_BRANCH: "${{ github.event.pull_request.base.ref }}"
+          HEAD_BRANCH: "${{ github.event.pull_request.head.ref }}"
         run: |
           # Format only the changed lines using clang-format-diff.py
           set -e
-          git diff -U0 --no-color origin/${{ env.BASE_BRANCH }}...origin/${{ env.HEAD_BRANCH }} -- . ':!generated/' > diff_output.patch
+          git diff -U0 --no-color origin/$BASE_BRANCH...origin/$HEAD_BRANCH -- . ':!generated/' > diff_output.patch
           cat diff_output.patch
           
       # run formatter on the differences if any

.gitignore

@@ -80,6 +80,8 @@ toolchains/android/
 
 # codegen
 tools/code-generation/generator/target/
+tools/code-generation/smithy/codegen/gradle/
+tools/code-generation/smithy/codegen/.gradle/
 
 #config output
 aws-cpp-sdk-core/include/aws/core/SDKConfig.h

CMakeLists.txt

@@ -67,6 +67,7 @@ if (LEGACY_BUILD)
     option(USE_TLS_V1_2 "Set http client to enforce TLS 1.2" ON)
     option(USE_TLS_V1_3 "Set http client to enforce TLS 1.3" OFF)
     option(ENABLE_SMOKE_TESTS "Enable smoke tests" OFF)
+    option(DISABLE_DNS_REQUIRED_TESTS "Disable unit tests that require DNS lookup to succeed, useful when using a http client that does not perform DNS lookup" OFF)
 
 
     set(AWS_USER_AGENT_CUSTOMIZATION "" CACHE STRING "User agent extension")
@@ -89,6 +90,10 @@ if (LEGACY_BUILD)
         add_definitions(-DENFORCE_TLS_V1_3)
     endif ()
 
+    if (DISABLE_DNS_REQUIRED_TESTS)
+        add_definitions(-DDISABLE_DNS_REQUIRED_TESTS)
+    endif ()
+
     #From https://stackoverflow.com/questions/18968979/how-to-get-colorized-output-with-cmake
     if (NOT WIN32)
         string(ASCII 27 Esc)

VERSION

@@ -1 +1 @@
-1.11.455
+1.11.479

cmake/sdksCommon.cmake

@@ -178,6 +178,7 @@ list(APPEND SDK_TEST_PROJECT_LIST "text-to-speech:tests/aws-cpp-sdk-text-to-spee
 list(APPEND SDK_TEST_PROJECT_LIST "transcribestreaming:tests/aws-cpp-sdk-transcribestreaming-integ-tests")
 list(APPEND SDK_TEST_PROJECT_LIST "eventbridge:tests/aws-cpp-sdk-eventbridge-tests")
 list(APPEND SDK_TEST_PROJECT_LIST "timestream-query:tests/aws-cpp-sdk-timestream-query-integration-tests")
+list(APPEND SDK_TEST_PROJECT_LIST "dsql:tests/aws-cpp-sdk-dsql-unit-tests")
 
 build_sdk_list()
 

generated/smoke-tests/acm/ACMSmokeTests.cpp

@@ -0,0 +1,61 @@
+/**
+ * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+ * SPDX-License-Identifier: Apache-2.0.
+ */
+#include <aws/core/utils/memory/AWSMemory.h>
+#include <aws/core/client/ClientConfiguration.h>
+#include <aws/core/client/CoreErrors.h>
+#include <aws/core/utils/UnreferencedParam.h>
+#include <utility>
+#include <aws/core/auth/AWSCredentialsProviderChain.h>
+#include <aws/core/http/HttpTypes.h>
+#include <aws/core/client/AsyncCallerContext.h>
+#include <aws/core/utils/Outcome.h>
+#include <aws/core/utils/logging/LogMacros.h>
+#include <algorithm>
+#include <aws/testing/AwsCppSdkGTestSuite.h>
+#include <aws/testing/AwsTestHelpers.h>
+#include <aws/acm/model/ListCertificatesRequest.h>
+#include <aws/acm/ACMClient.h>
+#include <aws/acm/model/GetCertificateRequest.h>
+
+namespace ACMSmokeTest{
+using namespace Aws::Auth;
+using namespace Aws::Http;
+using namespace Aws::Client;
+
+using namespace Aws::ACM;
+using namespace Aws::ACM::Model;
+class ACMSmokeTestSuite : public Aws::Testing::AwsCppSdkGTestSuite {
+    public:
+    static const char ALLOCATION_TAG[];
+};
+const char ACMSmokeTestSuite::ALLOCATION_TAG[] = "ACMSmokeTest";
+TEST_F(ACMSmokeTestSuite, GetCertificateFailure )
+{
+    Aws::ACM::ACMClientConfiguration clientConfiguration;
+    clientConfiguration.region = "us-west-2";
+    clientConfiguration.useFIPS = false;
+    clientConfiguration.useDualStack = false;
+    auto clientSp = Aws::MakeShared<ACMClient>(ALLOCATION_TAG, clientConfiguration);
+    //populate input params
+
+    GetCertificateRequest input;
+    input.SetCertificateArn("arn:aws:acm:region:123456789012:certificate/12345678-1234-1234-1234-123456789012");
+    auto outcome = clientSp->GetCertificate(input);
+    EXPECT_FALSE( outcome.IsSuccess());
+}
+TEST_F(ACMSmokeTestSuite, ListCertificatesSuccess )
+{
+    Aws::ACM::ACMClientConfiguration clientConfiguration;
+    clientConfiguration.region = "us-west-2";
+    clientConfiguration.useFIPS = false;
+    clientConfiguration.useDualStack = false;
+    auto clientSp = Aws::MakeShared<ACMClient>(ALLOCATION_TAG, clientConfiguration);
+    //populate input params
+
+    ListCertificatesRequest input;
+    auto outcome = clientSp->ListCertificates(input);
+    EXPECT_TRUE( outcome.IsSuccess());
+}
+}

generated/smoke-tests/acm/CMakeLists.txt

@@ -0,0 +1,41 @@
+add_project(acm-smoke-tests
+"Tests for the AWS ACM C++ SDK"
+testing-resources
+aws-cpp-sdk-acm
+aws-cpp-sdk-core
+)
+file(GLOB AWS_ACM_GENERATED_SMOKE_TEST_SRC
+"${CMAKE_CURRENT_SOURCE_DIR}/../RunTests.cpp"
+"${CMAKE_CURRENT_SOURCE_DIR}/*.cpp"
+)
+if(MSVC AND BUILD_SHARED_LIBS)
+    add_definitions(-DGTEST_LINKED_AS_SHARED_LIBRARY=1)
+endif()
+
+if (CMAKE_CROSSCOMPILING)
+    set(AUTORUN_UNIT_TESTS OFF)
+endif()
+
+if (AUTORUN_UNIT_TESTS)
+    enable_testing()
+endif()
+
+if(PLATFORM_ANDROID AND BUILD_SHARED_LIBS)
+    add_library(${PROJECT_NAME} "${AWS_ACM_GENERATED_SMOKE_TEST_SRC}")
+else()
+    add_executable(${PROJECT_NAME} "${AWS_ACM_GENERATED_SMOKE_TEST_SRC}")
+endif()
+
+set_compiler_flags(${PROJECT_NAME})
+set_compiler_warnings(${PROJECT_NAME})
+
+target_include_directories(${PROJECT_NAME} PUBLIC
+    ${CMAKE_CURRENT_SOURCE_DIR}/../../src/aws-cpp-sdk-acm/include)
+
+target_link_libraries(${PROJECT_NAME}
+    ${PROJECT_LIBS})
+
+if(NOT CMAKE_CROSSCOMPILING)
+    SET_TARGET_PROPERTIES(${PROJECT_NAME} PROPERTIES OUTPUT_NAME ${PROJECT_NAME})
+endif()
+

generated/smoke-tests/apigateway/APIGatewaySmokeTests.cpp

@@ -0,0 +1,46 @@
+/**
+ * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+ * SPDX-License-Identifier: Apache-2.0.
+ */
+#include <aws/core/utils/memory/AWSMemory.h>
+#include <aws/core/client/ClientConfiguration.h>
+#include <aws/core/client/CoreErrors.h>
+#include <aws/core/utils/UnreferencedParam.h>
+#include <utility>
+#include <aws/core/auth/AWSCredentialsProviderChain.h>
+#include <aws/core/http/HttpTypes.h>
+#include <aws/core/client/AsyncCallerContext.h>
+#include <aws/core/utils/Outcome.h>
+#include <aws/core/utils/logging/LogMacros.h>
+#include <algorithm>
+#include <aws/testing/AwsCppSdkGTestSuite.h>
+#include <aws/testing/AwsTestHelpers.h>
+#include <aws/apigateway/model/GetDomainNamesRequest.h>
+#include <aws/apigateway/APIGatewayClient.h>
+
+namespace APIGatewaySmokeTest{
+using namespace Aws::Auth;
+using namespace Aws::Http;
+using namespace Aws::Client;
+
+using namespace Aws::APIGateway;
+using namespace Aws::APIGateway::Model;
+class APIGatewaySmokeTestSuite : public Aws::Testing::AwsCppSdkGTestSuite {
+    public:
+    static const char ALLOCATION_TAG[];
+};
+const char APIGatewaySmokeTestSuite::ALLOCATION_TAG[] = "APIGatewaySmokeTest";
+TEST_F(APIGatewaySmokeTestSuite, GetDomainNamesSuccess )
+{
+    Aws::APIGateway::APIGatewayClientConfiguration clientConfiguration;
+    clientConfiguration.region = "us-west-2";
+    clientConfiguration.useFIPS = false;
+    clientConfiguration.useDualStack = false;
+    auto clientSp = Aws::MakeShared<APIGatewayClient>(ALLOCATION_TAG, clientConfiguration);
+    //populate input params
+
+    GetDomainNamesRequest input;
+    auto outcome = clientSp->GetDomainNames(input);
+    EXPECT_TRUE( outcome.IsSuccess());
+}
+}

generated/smoke-tests/apigateway/CMakeLists.txt

@@ -0,0 +1,41 @@
+add_project(apigateway-smoke-tests
+"Tests for the AWS APIGATEWAY C++ SDK"
+testing-resources
+aws-cpp-sdk-apigateway
+aws-cpp-sdk-core
+)
+file(GLOB AWS_APIGATEWAY_GENERATED_SMOKE_TEST_SRC
+"${CMAKE_CURRENT_SOURCE_DIR}/../RunTests.cpp"
+"${CMAKE_CURRENT_SOURCE_DIR}/*.cpp"
+)
+if(MSVC AND BUILD_SHARED_LIBS)
+    add_definitions(-DGTEST_LINKED_AS_SHARED_LIBRARY=1)
+endif()
+
+if (CMAKE_CROSSCOMPILING)
+    set(AUTORUN_UNIT_TESTS OFF)
+endif()
+
+if (AUTORUN_UNIT_TESTS)
+    enable_testing()
+endif()
+
+if(PLATFORM_ANDROID AND BUILD_SHARED_LIBS)
+    add_library(${PROJECT_NAME} "${AWS_APIGATEWAY_GENERATED_SMOKE_TEST_SRC}")
+else()
+    add_executable(${PROJECT_NAME} "${AWS_APIGATEWAY_GENERATED_SMOKE_TEST_SRC}")
+endif()
+
+set_compiler_flags(${PROJECT_NAME})
+set_compiler_warnings(${PROJECT_NAME})
+
+target_include_directories(${PROJECT_NAME} PUBLIC
+    ${CMAKE_CURRENT_SOURCE_DIR}/../../src/aws-cpp-sdk-apigateway/include)
+
+target_link_libraries(${PROJECT_NAME}
+    ${PROJECT_LIBS})
+
+if(NOT CMAKE_CROSSCOMPILING)
+    SET_TARGET_PROPERTIES(${PROJECT_NAME} PROPERTIES OUTPUT_NAME ${PROJECT_NAME})
+endif()
+

generated/smoke-tests/application-autoscaling/ApplicationAutoScalingSmokeTests.cpp

@@ -0,0 +1,47 @@
+/**
+ * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
+ * SPDX-License-Identifier: Apache-2.0.
+ */
+#include <aws/core/utils/memory/AWSMemory.h>
+#include <aws/core/client/ClientConfiguration.h>
+#include <aws/core/client/CoreErrors.h>
+#include <aws/core/utils/UnreferencedParam.h>
+#include <utility>
+#include <aws/core/auth/AWSCredentialsProviderChain.h>
+#include <aws/core/http/HttpTypes.h>
+#include <aws/core/client/AsyncCallerContext.h>
+#include <aws/core/utils/Outcome.h>
+#include <aws/core/utils/logging/LogMacros.h>
+#include <algorithm>
+#include <aws/testing/AwsCppSdkGTestSuite.h>
+#include <aws/testing/AwsTestHelpers.h>
+#include <aws/application-autoscaling/model/DescribeScalableTargetsRequest.h>
+#include <aws/application-autoscaling/ApplicationAutoScalingClient.h>
+
+namespace ApplicationAutoScalingSmokeTest{
+using namespace Aws::Auth;
+using namespace Aws::Http;
+using namespace Aws::Client;
+
+using namespace Aws::ApplicationAutoScaling;
+using namespace Aws::ApplicationAutoScaling::Model;
+class ApplicationAutoScalingSmokeTestSuite : public Aws::Testing::AwsCppSdkGTestSuite {
+    public:
+    static const char ALLOCATION_TAG[];
+};
+const char ApplicationAutoScalingSmokeTestSuite::ALLOCATION_TAG[] = "ApplicationAutoScalingSmokeTest";
+TEST_F(ApplicationAutoScalingSmokeTestSuite, DescribeScalableTargetsSuccess )
+{
+    Aws::ApplicationAutoScaling::ApplicationAutoScalingClientConfiguration clientConfiguration;
+    clientConfiguration.region = "us-west-2";
+    clientConfiguration.useFIPS = false;
+    clientConfiguration.useDualStack = false;
+    auto clientSp = Aws::MakeShared<ApplicationAutoScalingClient>(ALLOCATION_TAG, clientConfiguration);
+    //populate input params
+
+    DescribeScalableTargetsRequest input;
+    input.SetServiceNamespace({ServiceNamespace::ec2});
+    auto outcome = clientSp->DescribeScalableTargets(input);
+    EXPECT_TRUE( outcome.IsSuccess());
+}
+}

generated/smoke-tests/application-autoscaling/CMakeLists.txt

@@ -0,0 +1,41 @@
+add_project(application-autoscaling-smoke-tests
+"Tests for the AWS APPLICATION-AUTOSCALING C++ SDK"
+testing-resources
+aws-cpp-sdk-application-autoscaling
+aws-cpp-sdk-core
+)
+file(GLOB AWS_APPLICATION-AUTOSCALING_GENERATED_SMOKE_TEST_SRC
+"${CMAKE_CURRENT_SOURCE_DIR}/../RunTests.cpp"
+"${CMAKE_CURRENT_SOURCE_DIR}/*.cpp"
+)
+if(MSVC AND BUILD_SHARED_LIBS)
+    add_definitions(-DGTEST_LINKED_AS_SHARED_LIBRARY=1)
+endif()
+
+if (CMAKE_CROSSCOMPILING)
+    set(AUTORUN_UNIT_TESTS OFF)
+endif()
+
+if (AUTORUN_UNIT_TESTS)
+    enable_testing()
+endif()
+
+if(PLATFORM_ANDROID AND BUILD_SHARED_LIBS)
+    add_library(${PROJECT_NAME} "${AWS_APPLICATION-AUTOSCALING_GENERATED_SMOKE_TEST_SRC}")
+else()
+    add_executable(${PROJECT_NAME} "${AWS_APPLICATION-AUTOSCALING_GENERATED_SMOKE_TEST_SRC}")
+endif()
+
+set_compiler_flags(${PROJECT_NAME})
+set_compiler_warnings(${PROJECT_NAME})
+
+target_include_directories(${PROJECT_NAME} PUBLIC
+    ${CMAKE_CURRENT_SOURCE_DIR}/../../src/aws-cpp-sdk-application-autoscaling/include)
+
+target_link_libraries(${PROJECT_NAME}
+    ${PROJECT_LIBS})
+
+if(NOT CMAKE_CROSSCOMPILING)
+    SET_TARGET_PROPERTIES(${PROJECT_NAME} PROPERTIES OUTPUT_NAME ${PROJECT_NAME})
+endif()
+