Add some minor fixes to logging (#686)

project-ncl · Aug 28, 2024 · 8d452dc · 8d452dc
1 parent 5b22dcd
commit 8d452dc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/main/java/org/jboss/pnc/deliverablesanalyzer/rest/AnalyzeResource.java b/src/main/java/org/jboss/pnc/deliverablesanalyzer/rest/AnalyzeResource.java
@@ -152,7 +152,7 @@ public Response analyze(AnalyzePayload analyzePayload) throws URISyntaxException
                 LOGGER.info(
                         "Analysis with ID {} was {} finished and callback was performed.",
                         id,
-                        analysisReport.isSuccess() ? "successfully" : "unsuccessfully");
+                        analysisReport != null && analysisReport.isSuccess() ? "successfully" : "unsuccessfully");
             } finally {
                 if (analyzePayload.getHeartbeat() != null) {
                     heartbeatScheduler.unsubscribeRequest(id);
@@ -224,10 +224,10 @@ private BuildConfig prepareConfig(String rawConfig) throws IOException {
     }
 
     /**
-     * Given a request and a map of http headers, add the http headers to the request if not already in the request
+     * Given a request and a map of HTTP headers, add the HTTP headers to the request if not already in the request
      *
-     * @param request
-     * @param httpHeaders
+     * @param request the request
+     * @param httpHeaders the HTTP headers
      */
     private static void mergeHttpHeaders(Request request, Map<String, String> httpHeaders) {
 

diff --git a/src/main/java/org/jboss/pnc/deliverablesanalyzer/rest/HttpClient.java b/src/main/java/org/jboss/pnc/deliverablesanalyzer/rest/HttpClient.java
@@ -86,7 +86,7 @@ public void performHttpRequest(Request request) throws Exception {
      * @throws IOException Thrown in case of the request failure
      */
     public void performHttpRequest(Request request, Object payload) throws Exception {
-        LOGGER.debug("Performing HTTP request with these parameters: {}", request);
+        LOGGER.debug("Performing HTTP request with these parameters: {} (payload: {})", request, payload);
 
         Response response = null;