Skip to content

Commit f505213

Browse files
committed
Fix checkstyle
1 parent 04e4e43 commit f505213

File tree

1 file changed

+16
-9
lines changed

1 file changed

+16
-9
lines changed

services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/handlers/ExceptionTranslationInterceptor.java

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -75,15 +75,22 @@ public Throwable modifyException(Context.FailedExecution context, ExecutionAttri
7575
.message(message)
7676
.build();
7777
}
78-
} else if (exception.statusCode() == 503 && "Slow Down".equals(errorDetails.sdkHttpResponse().statusText().orElse(null))) {
79-
return S3Exception.builder()
80-
.awsErrorDetails(fillErrorDetails(errorDetails, "SlowDown", "Please reduce your request rate."))
81-
.statusCode(503)
82-
.requestId(requestId)
83-
.extendedRequestId(extendedRequestId)
84-
.message(message)
85-
.build();
86-
} else if (errorDetails.errorMessage() == null) {
78+
}
79+
80+
if (exception.statusCode() == 503) {
81+
if ("Slow Down".equals(errorDetails.sdkHttpResponse().statusText().orElse(null))) {
82+
return S3Exception.builder()
83+
.awsErrorDetails(fillErrorDetails(errorDetails, "SlowDown",
84+
"Please reduce your request rate."))
85+
.statusCode(503)
86+
.requestId(requestId)
87+
.extendedRequestId(extendedRequestId)
88+
.message(message)
89+
.build();
90+
}
91+
}
92+
93+
if (errorDetails.errorMessage() == null) {
8794
// Populate the error message using the HTTP response status text. Usually that's just the value from the
8895
// HTTP spec (e.g. "Forbidden"), but sometimes S3 throws some more useful things in there, like "Slow Down".
8996
String errorMessage = errorDetails.sdkHttpResponse().statusText().orElse(null);

0 commit comments

Comments
 (0)