Move filter to a set, add user-agent to unsigned headers

Author: Bret Ambrose

aws-cpp-sdk-core/include/aws/core/auth/AWSAuthSigner.h

@@ -19,6 +19,7 @@
 
 #include <aws/core/Region.h>
 #include <aws/core/utils/memory/AWSMemory.h>
+#include <aws/core/utils/memory/stl/AWSSet.h>
 #include <aws/core/utils/DateTime.h>
 #include <aws/core/utils/Array.h>
 
@@ -130,13 +131,16 @@ namespace Aws
             Aws::String GenerateStringToSign(const Aws::String& dateValue, const Aws::String& simpleDate, const Aws::String& canonicalRequestHash) const;
             const Aws::Utils::ByteBuffer& ComputeLongLivedHash(const Aws::String& secretKey, const Aws::String& simpleDate) const;
 
-            static bool ShouldSignHeader(const Aws::String& header);
+            bool ShouldSignHeader(const Aws::String& header) const;
 
             std::shared_ptr<Auth::AWSCredentialsProvider> m_credentialsProvider;
             Aws::String m_serviceName;
             Aws::String m_region;
             Aws::UniquePtr<Aws::Utils::Crypto::Sha256> m_hash;
             Aws::UniquePtr<Aws::Utils::Crypto::Sha256HMAC> m_HMAC;
+
+            Aws::Set<Aws::String> m_unsignedHeaders;
+
             //these next four fields are ONLY for caching purposes and do not change
             //the logical state of the signer. They are marked mutable so the
             //interface can remain const.

aws-cpp-sdk-core/source/auth/AWSAuthSigner.cpp

@@ -129,6 +129,7 @@ AWSAuthV4Signer::AWSAuthV4Signer(const std::shared_ptr<Auth::AWSCredentialsProvi
     m_region(region),
     m_hash(Aws::MakeUnique<Aws::Utils::Crypto::Sha256>(v4LogTag)),
     m_HMAC(Aws::MakeUnique<Aws::Utils::Crypto::Sha256HMAC>(v4LogTag)),
+    m_unsignedHeaders({"user-agent", "x-amzn-trace-id"}),
     m_signPayloads(signPayloads),
     m_urlEscapePath(urlEscapePath)
 {
@@ -141,10 +142,10 @@ AWSAuthV4Signer::~AWSAuthV4Signer()
     // empty destructor in .cpp file to keep from needing the implementation of (AWSCredentialsProvider, Sha256, Sha256HMAC) in the header file 
 }
 
-// If this ever grows, convert to a static hash set initialized on InitAPI
-bool AWSAuthV4Signer::ShouldSignHeader(const Aws::String& header)
+
+bool AWSAuthV4Signer::ShouldSignHeader(const Aws::String& header) const
 {
-    return Aws::Utils::StringUtils::ToLower(header.c_str()) != Aws::String("x-amzn-trace-id");
+    return m_unsignedHeaders.find(Aws::Utils::StringUtils::ToLower(header.c_str())) == m_unsignedHeaders.cend();
 }
 
 bool AWSAuthV4Signer::SignRequest(Aws::Http::HttpRequest& request) const