Merge pull request #248 from checkmarx-ltd/2024_Q1_Integration_Branch

Jenkins Plugin Release Plugin Version 2024.1.2
checkmarx-ltd · Apr 12, 2024 · 3b9c0d3 · 3b9c0d3
2 parents f48efbc + 15357f9
commit 3b9c0d3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/main/java/com/cx/restclient/ast/AstScaClient.java b/src/main/java/com/cx/restclient/ast/AstScaClient.java
@@ -308,8 +308,8 @@ public Results waitForScanResults() {
 				}
 
 				fileName = PDF_REPORT_NAME + "_" + now + "." + reportFormat.toLowerCase();
-				String pdfLink = SASTUtils.writePDFReport(scanReport, config.getReportsDir(), fileName, log);
 				if (reportFormat.toLowerCase().equals("pdf")) {
+				String pdfLink = SASTUtils.writePDFReport(scanReport, config.getReportsDir(), fileName, log);
 					scaResults.setScaPDFLink(pdfLink);
 					scaResults.setPdfFileName(fileName);
 				}

diff --git a/src/main/java/com/cx/restclient/sast/utils/LegacyClient.java b/src/main/java/com/cx/restclient/sast/utils/LegacyClient.java
@@ -126,6 +126,9 @@ public long resolveProjectId() throws IOException {
 			}
         } else {
             projectId = projects.get(0).getId();
+            if(config.isEnableDataRetention()) {
+                setRetentionRate(projectId);
+            }
             setIsNewProject(false);
             log.info("Project already exists with ID {}", projectId);
         }