feat(get): include request/response headers in debugs

src/main/java/me/itzg/helpers/get/GetCommand.java

@@ -388,6 +388,7 @@ private Path processSingleUri(URI uri, CloseableHttpClient client,
 
         final Path file;
         try {
+            log.debug("Executing {} with headers={}", request, request.getHeaders());
             file = client.execute(request, handler);
         } catch (HttpResponseException e) {
             throw new FailedToDownloadException(uri, e);

src/main/java/me/itzg/helpers/get/JsonPathOutputHandler.java

@@ -11,10 +11,9 @@
 import java.nio.file.Path;
 import java.util.List;
 import java.util.stream.Collectors;
-import org.apache.hc.client5.http.impl.classic.AbstractHttpClientResponseHandler;
 import org.apache.hc.core5.http.HttpEntity;
 
-class JsonPathOutputHandler extends AbstractHttpClientResponseHandler<Path> implements OutputResponseHandler {
+class JsonPathOutputHandler extends LoggingResponseHandler<Path> implements OutputResponseHandler {
 
   private final PrintWriter writer;
   private final String jsonPath;

src/main/java/me/itzg/helpers/get/LoggingResponseHandler.java

@@ -0,0 +1,22 @@
+package me.itzg.helpers.get;
+
+import java.io.IOException;
+import lombok.extern.slf4j.Slf4j;
+import org.apache.hc.client5.http.impl.classic.AbstractHttpClientResponseHandler;
+import org.apache.hc.core5.http.ClassicHttpResponse;
+import org.slf4j.Logger;
+
+@Slf4j
+public abstract class LoggingResponseHandler<T> extends AbstractHttpClientResponseHandler<T> {
+
+  static void logResponse(Logger log, ClassicHttpResponse response) {
+    log.debug("Response: status={}, reason={}, headers={}",
+        response.getCode(), response.getReasonPhrase(), response.getHeaders());
+  }
+
+  @Override
+  public T handleResponse(ClassicHttpResponse response) throws IOException {
+    logResponse(log, response);
+    return super.handleResponse(response);
+  }
+}

src/main/java/me/itzg/helpers/get/NotModifiedHandler.java

@@ -21,6 +21,8 @@ public NotModifiedHandler(Path file, OutputResponseHandler delegate, boolean log
 
   @Override
   public Path handleResponse(ClassicHttpResponse response) throws HttpException, IOException {
+    LoggingResponseHandler.logResponse(log, response);
+
     if (response.getCode() == HttpStatus.SC_NOT_MODIFIED) {
       if (logProgressEach) {
         log.info("Skipping {} since it is already up to date", file);

src/main/java/me/itzg/helpers/get/OutputToDirectoryHandler.java

@@ -24,6 +24,8 @@ public OutputToDirectoryHandler(Path directory,
 
   @Override
   public Path handleResponse(ClassicHttpResponse response) throws HttpException, IOException {
+    LoggingResponseHandler.logResponse(log, response);
+
     final String filename = filenameExtractor.extract(response);
 
     final Path filePath = directory.resolve(filename);

src/main/java/me/itzg/helpers/get/PrintWriterHandler.java

@@ -3,10 +3,9 @@
 import java.io.IOException;
 import java.io.PrintWriter;
 import java.nio.file.Path;
-import org.apache.hc.client5.http.impl.classic.AbstractHttpClientResponseHandler;
 import org.apache.hc.core5.http.HttpEntity;
 
-class PrintWriterHandler extends AbstractHttpClientResponseHandler<Path> implements OutputResponseHandler {
+class PrintWriterHandler extends LoggingResponseHandler<Path> implements OutputResponseHandler {
   private final PrintWriter writer;
 
   public PrintWriterHandler(PrintWriter writer) {

src/main/java/me/itzg/helpers/get/SaveToFileHandler.java

@@ -5,11 +5,10 @@
 import java.nio.file.Files;
 import java.nio.file.Path;
 import lombok.extern.slf4j.Slf4j;
-import org.apache.hc.client5.http.impl.classic.AbstractHttpClientResponseHandler;
 import org.apache.hc.core5.http.HttpEntity;
 
 @Slf4j
-public class SaveToFileHandler extends AbstractHttpClientResponseHandler<Path> implements OutputResponseHandler {
+public class SaveToFileHandler extends LoggingResponseHandler<Path> implements OutputResponseHandler {
 
   private final Path outputFile;
   private final boolean logProgressEach;