Merge pull request #312 from RachelTucker/response_logging

Improved network client logging to include request id

Author: Denver

ds3-sdk/src/main/java/com/spectralogic/ds3client/networking/NetworkClientImpl.java

@@ -82,6 +82,7 @@ public class NetworkClientImpl implements NetworkClient {
     final static private String CONTENT_CRC32C = "Content-CRC32C";
     final static private int MAX_CONNECTION_PER_ROUTE = 50;
     final static private int MAX_CONNECTION_TOTAL = 100;
+    final static private String REQUEST_ID_HEADER = "x-amz-request-id";
 
     final private ConnectionDetails connectionDetails;
 
@@ -187,13 +188,21 @@ public WebResponse getResponse(final Ds3Request request) throws IOException {
             int redirectCount = 0;
             do {
                 final CloseableHttpResponse response = requestExecutor.execute();
+                String requestId = "Unknown";
+                if (response.containsHeader(REQUEST_ID_HEADER)) {
+                    requestId = response.getFirstHeader(REQUEST_ID_HEADER).getValue();
+                }
                 if (response.getStatusLine().getStatusCode() == HttpStatus.SC_TEMPORARY_REDIRECT) {
                     redirectCount++;
+                    LOG.info("Performing retry - attempt: {} for request #{}",
+                            redirectCount,
+                            requestId);
                     response.close();
-                    LOG.info("Performing retry - attempt: {}", redirectCount);
                 }
                 else {
-                    LOG.info("Got response from server");
+                    LOG.info("Server responded with {} for request #{}",
+                            response.getStatusLine().getStatusCode(),
+                            requestId);
                     return new WebResponseImpl(response);
                 }
             } while (redirectCount < this.connectionDetails.getRetries());