Skip to content

Commit cff5de0

Browse files
fixed sonarcube issues
1 parent 8a3a25e commit cff5de0

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/presignedurl/DefaultPresignedUrlManager.java

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ public <ReturnT> ReturnT getObject(PresignedUrlGetObjectRequest presignedUrlGetO
8787
.range(presignedUrlGetObjectRequest.range())
8888
.build();
8989

90-
SdkClientConfiguration clientConfiguration = updateSdkClientConfiguration(internalRequest, this.clientConfiguration);
90+
SdkClientConfiguration clientConfiguration = updateSdkClientConfiguration(this.clientConfiguration);
9191
List<MetricPublisher> metricPublishers = Optional.ofNullable(
9292
clientConfiguration.option(SdkClientOption.METRIC_PUBLISHERS))
9393
.orElse(Collections.emptyList());
@@ -115,12 +115,11 @@ public <ReturnT> ReturnT getObject(PresignedUrlGetObjectRequest presignedUrlGetO
115115
}
116116
}
117117

118-
private SdkClientConfiguration updateSdkClientConfiguration(PresignedUrlGetObjectRequestWrapper request,
119-
SdkClientConfiguration clientConfiguration) {
120-
SdkClientConfiguration.Builder configuration = clientConfiguration.toBuilder();
121-
configuration.option(SdkAdvancedClientOption.SIGNER, new NoOpSigner());
122-
configuration.option(SIGNER_OVERRIDDEN, true);
123-
return configuration.build();
118+
private SdkClientConfiguration updateSdkClientConfiguration(SdkClientConfiguration configuration) {
119+
SdkClientConfiguration.Builder configBuilder = clientConfiguration.toBuilder();
120+
configBuilder.option(SdkAdvancedClientOption.SIGNER, new NoOpSigner());
121+
configBuilder.option(SIGNER_OVERRIDDEN, true);
122+
return configBuilder.build();
124123
}
125124

126125
}

0 commit comments

Comments
 (0)