fix(region_config): remove mutation of global signatureVersion (#4288)

* fix(region_config): remove mutation of global signatureVersion * Update .changes/next-release/bugfix-region-config-31f590e0.json Co-authored-by: Trivikram Kamat <16024985+trivikr@users.noreply.github.com>
aws · Nov 28, 2022 · c6c5aab · c6c5aab
1 parent b168eaa
commit c6c5aab
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/.changes/next-release/bugfix-region-config-31f590e0.json b/.changes/next-release/bugfix-region-config-31f590e0.json
@@ -0,0 +1,5 @@
+{
+  "type": "bugfix",
+  "category": "region_config",
+  "description": "Set signatureVersion to bearer explcitly when defined in service API"
+}
diff --git a/lib/region_config.js b/lib/region_config.js
@@ -64,11 +64,20 @@ function configureEndpoint(service) {
 
       // signature version
       if (!config.signatureVersion) {
-        config.signatureVersion = (service.api && service.api.signatureVersion) || 'v4';
+        // Note: config is a global object and should not be mutated here.
+        // However, we are retaining this line for backwards compatibility.
+        // The non-v4 signatureVersion will be set in a copied object below.
+        config.signatureVersion = 'v4';
       }
 
+      var useBearer = (service.api && service.api.signatureVersion) === 'bearer';
+
       // merge config
-      applyConfig(service, config);
+      applyConfig(service, Object.assign(
+        {},
+        config,
+        { signatureVersion: useBearer ? 'bearer' : config.signatureVersion }
+      ));
       return;
     }
   }