Skip to content

Commit 5c28a19

Browse files
authored
Fixed an issue where using S3ClientConfiguration.Builder.endpointOverride could cause AWS_ERROR_INVALID_ARGUMENT. (#3315)
1 parent 2b57b54 commit 5c28a19

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
{
2+
"type": "bugfix",
3+
"category": "Amazon S3 Transfer Manager",
4+
"contributor": "",
5+
"description": "Fixed an issue where using S3ClientConfiguration.Builder.endpointOverride could cause AWS_ERROR_INVALID_ARGUMENT."
6+
}

services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/crt/S3CrtAsyncHttpClient.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,7 @@ public CompletableFuture<Void> execute(AsyncExecuteRequest asyncRequest) {
9191
S3MetaRequestOptions requestOptions = new S3MetaRequestOptions()
9292
.withHttpRequest(httpRequest)
9393
.withMetaRequestType(requestType)
94-
.withResponseHandler(responseHandler)
95-
.withEndpoint(s3NativeClientConfiguration.endpointOverride());
94+
.withResponseHandler(responseHandler);
9695

9796
try (S3MetaRequest s3MetaRequest = crtS3Client.makeMetaRequest(requestOptions)) {
9897
closeResourcesWhenComplete(executeFuture, s3MetaRequest, responseHandler);

services/s3/src/test/java/software/amazon/awssdk/services/s3/internal/crt/S3CrtAsyncHttpClientTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,6 @@ public void defaultRequest_shouldSetMetaRequestOptionsCorrectly() {
8484
S3MetaRequestOptions actual = s3MetaRequestOptionsArgumentCaptor.getValue();
8585
assertThat(actual.getMetaRequestType()).isEqualTo(S3MetaRequestOptions.MetaRequestType.DEFAULT);
8686
assertThat(actual.getCredentialsProvider()).isNull();
87-
assertThat(actual.getEndpoint().equals(DEFAULT_ENDPOINT));
8887

8988
HttpRequest httpRequest = actual.getHttpRequest();
9089
assertThat(httpRequest.getEncodedPath()).isEqualTo("/key");

0 commit comments

Comments
 (0)