Response can be null in afterError of LoggingAwareRequestHandler in S3

This commit is contained in:
Julian Raufelder 2021-04-30 11:51:32 +02:00
parent 3e36de814d
commit 50fbec1bae
No known key found for this signature in database
GPG Key ID: 17EE71F6634E381D

View File

@ -72,12 +72,16 @@ class S3ClientFactory {
@Override @Override
public void afterError(Request<?> request, Response<?> response, Exception e) { public void afterError(Request<?> request, Response<?> response, Exception e) {
Timber.tag("S3Client").e( // if (response != null) {
e, Timber.tag("S3Client").e( //
"Error occurred (%s) with status %s (%s)", // e, //
request.getAWSRequestMetrics().getTimingInfo().getStartTimeNano(), // "Error occurred (%s) with status %s (%s)", //
response.getHttpResponse().getStatusText(), // request.getAWSRequestMetrics().getTimingInfo().getStartTimeNano(), //
response.getHttpResponse().getStatusCode()); response.getHttpResponse().getStatusText(), //
response.getHttpResponse().getStatusCode());
} else {
Timber.tag("S3Client").e(e, "Error occurred (%s)", request.getAWSRequestMetrics().getTimingInfo().getStartTimeNano());
}
} }
} }
} }