fix: check for vulnerabilities presence

mesosphere · Feb 19, 2024 · e548b82 · e548b82
1 parent 8ba7f98
commit e548b82
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/.github/actions/copacetic-action/pkg/image/scan.go b/.github/actions/copacetic-action/pkg/image/scan.go
@@ -22,6 +22,14 @@ func (r *Report) WriteTo(path string) error {
 	return os.WriteFile(path, data, 0o755)
 }
 
+func (r *Report) Vulnerabilities() []types.DetectedVulnerability {
+	vulnerabilities := []types.DetectedVulnerability{}
+	for _, resultClass := range r.Results {
+		vulnerabilities = append(vulnerabilities, resultClass.Vulnerabilities...)
+	}
+	return vulnerabilities
+}
+
 type CmdErr struct {
 	Err    error
 	Stdout []byte

diff --git a/.github/actions/copacetic-action/pkg/patch/task.go b/.github/actions/copacetic-action/pkg/patch/task.go
@@ -54,12 +54,12 @@ func Run(ctx context.Context, imageRef string, reg registry.Registry, imageTagSu
 		return withErr(t, err), err
 	}
 
-	if len(report.Results) == 0 {
+	if len(report.Vulnerabilities()) == 0 {
 		logger.Info("no fixable vulnerabilities found in scanned image", "scannedImage", imagePatch.Scanned)
 		return t, nil
 	}
 
-	logger.Info("found patchable vulnerabilities", "report", report.Results)
+	logger.Info("found patchable vulnerabilities", "vulnerabilites", report.Vulnerabilities())
 
 	buildId, err := randutil.Alphanumeric(5)
 	logger.Info("generated unique buildId", "buildId", buildId)