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 <[email protected]>

Author: kuhe

.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"
+}

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;
     }
   }