Skip to content

Commit e8f040e

Browse files
authored
fix using accesspoint in ManagedUpload (#3109)
1 parent df4cdfc commit e8f040e

File tree

3 files changed

+53
-2
lines changed

3 files changed

+53
-2
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
{
2+
"type": "bugfix",
3+
"category": "Access Point",
4+
"description": "Avoid mixing up S3 client config set by ManagedUpload with user-set client config"
5+
}

lib/s3/managed_upload.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -284,7 +284,7 @@ AWS.S3.ManagedUpload = AWS.util.inherit({
284284
self.service = new AWS.S3({params: params});
285285
} else {
286286
var service = self.service;
287-
var config = AWS.util.copy(service.config);
287+
var config = AWS.util.copy(service._originalConfig || {});
288288
config.signatureVersion = service.getSignatureVersion();
289289
self.service = new service.constructor.__super__(config);
290290
self.service.config.params =

test/s3/managed_upload.spec.js

+47-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)