Fix for pdf report not using default engines (#587)

* fix for pdf report not using default engines * fix for pdf report not using default engines * Update one-scan.yml * Update one-scan.yml --------- Co-authored-by: Pedro Lopes <[email protected]>
Checkmarx · Sep 29, 2023 · 6790ba2 · 6790ba2
1 parent f4271fe
commit 6790ba2
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/internal/commands/result.go b/internal/commands/result.go
@@ -1109,11 +1109,11 @@ func exportSbomResults(sbomWrapper wrappers.ResultsSbomWrapper,
 func exportPdfResults(pdfWrapper wrappers.ResultsPdfWrapper, summary *wrappers.ResultSummary, summaryRpt, formatPdfToEmail, pdfOptions string) error {
 	pdfReportsPayload := &wrappers.PdfReportsPayload{}
 	pollingResp := &wrappers.PdfPollingResponse{}
-
-	pdfOptionsSections, pdfOptionsEngines, err := validatePdfOptions(pdfOptions)
+	pdfOptionsSections, pdfOptionsEngines, err := parsePDFOptions(pdfOptions, summary.EnginesEnabled)
 	if err != nil {
 		return err
 	}
+
 	pdfReportsPayload.ReportName = reportNameScanReport
 	pdfReportsPayload.ReportType = "cli"
 	pdfReportsPayload.FileFormat = printer.FormatPDF
@@ -1178,7 +1178,7 @@ func validateSbomOptions(sbomOption string) (string, error) {
 	return "", errors.Errorf("invalid SBOM option: %s", sbomOption)
 }
 
-func validatePdfOptions(pdfOptions string) (pdfOptionsSections, pdfOptionsEngines []string, err error) {
+func parsePDFOptions(pdfOptions string, enabledEngines []string) (pdfOptionsSections, pdfOptionsEngines []string, err error) {
 	var pdfOptionsSectionsMap = map[string]string{
 		"scansummary":      "ScanSummary",
 		"executivesummary": "ExecutiveSummary",
@@ -1201,6 +1201,13 @@ func validatePdfOptions(pdfOptions string) (pdfOptionsSections, pdfOptionsEngine
 			return nil, nil, errors.Errorf("report option \"%s\" unavailable", s)
 		}
 	}
+	if pdfOptionsEngines == nil {
+		for _, engine := range enabledEngines {
+			if pdfOptionsEnginesMap[engine] != "" {
+				pdfOptionsEngines = append(pdfOptionsEngines, pdfOptionsEnginesMap[engine])
+			}
+		}
+	}
 	return pdfOptionsSections, pdfOptionsEngines, nil
 }