Add transfer-encoding and connection to list of unsigned sigv4 headers (#3176)

alextwoods · web-flow · commit 539e65b965d7 · 2025-01-30T09:23:59.000-08:00
diff --git a/gems/aws-sdk-core/CHANGELOG.md b/gems/aws-sdk-core/CHANGELOG.md
@@ -1,6 +1,8 @@
 Unreleased Changes
 ------------------
 
+* Issue - Add `transfer-encoding` and `connection` to list of unsigned sigv4 headers.
+
 3.217.0 (2025-01-24)
 ------------------
 
diff --git a/gems/aws-sdk-core/lib/aws-sdk-core/plugins/sign.rb b/gems/aws-sdk-core/lib/aws-sdk-core/plugins/sign.rb
@@ -113,7 +113,7 @@ def initialize(auth_scheme, config, sigv4_overrides = {})
               signing_algorithm: scheme_name.to_sym,
               uri_escape_path: !!!auth_scheme['disableDoubleEncoding'],
               normalize_path: !!!auth_scheme['disableNormalizePath'],
-              unsigned_headers: %w[content-length user-agent x-amzn-trace-id]
+              unsigned_headers: %w[content-length user-agent x-amzn-trace-id expect transfer-encoding connection]
             )
           rescue Aws::Sigv4::Errors::MissingCredentialsError
             raise Aws::Errors::MissingCredentialsError

Original file line number	Diff line number	Diff line change
`@@ -113,7 +113,7 @@ def initialize(auth_scheme, config, sigv4_overrides = {})`
`113`	`113`	`signing_algorithm: scheme_name.to_sym,`
`114`	`114`	`uri_escape_path: !!!auth_scheme['disableDoubleEncoding'],`
`115`	`115`	`normalize_path: !!!auth_scheme['disableNormalizePath'],`
`116`		`- unsigned_headers: %w[content-length user-agent x-amzn-trace-id]`
	`116`	`+ unsigned_headers: %w[content-length user-agent x-amzn-trace-id expect transfer-encoding connection]`
`117`	`117`	`)`
`118`	`118`	`rescue Aws::Sigv4::Errors::MissingCredentialsError`
`119`	`119`	`raise Aws::Errors::MissingCredentialsError`