Merge pull request #209 from GetFeedback/add-logs-api-client

Add error log in case of IOException in ApiClient
GetFeedback · Sep 16, 2022 · 232e150 · 232e150
2 parents a21be10 + 8a74c25
commit 232e150
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/config/dependency-check-suppression.xml b/config/dependency-check-suppression.xml
@@ -48,5 +48,8 @@
         <packageUrl regex="true">^pkg:maven/org\.yaml/snakeyaml@.*$</packageUrl>
         <cve>CVE-2022-25857</cve>
         <cve>CVE-2022-38752</cve>
+        <cve>CVE-2022-38749</cve>
+        <cve>CVE-2022-38751</cve>
+        <cve>CVE-2022-38750</cve>
     </suppress>
 </suppressions>
diff --git a/...toconfigure/src/main/java/dev/vox/platform/kahpp/configuration/http/client/ApiClient.java b/...toconfigure/src/main/java/dev/vox/platform/kahpp/configuration/http/client/ApiClient.java
@@ -29,9 +29,13 @@
 import org.apache.http.impl.conn.SystemDefaultRoutePlanner;
 import org.apache.http.message.BasicHeader;
 import org.apache.http.util.EntityUtils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public final class ApiClient {
 
+  private static final Logger LOGGER = LoggerFactory.getLogger(ApiClient.class);
+
   private final HttpClient httpClient;
   private final String basePath;
   private final Map<String, String> defaultHeaders;
@@ -59,6 +63,7 @@ private Response executeRequest(final Request request) throws RequestException {
     try {
       response = transformHttpResponse(httpClient.execute(httpRequest));
     } catch (IOException exception) {
+      LOGGER.error("Request {} failed because {}", httpRequest, exception.getMessage());
       throw new TransferException(exception, request);
     }