Skip to content

Commit d1da33d

Browse files
committed
validate: Fixes G104-gosec - handle errors in rescheduleScan and purgeOldClusterScanReports
*pkg/securityscan/jobHandler.go:72:5: G104: Errors unhandled (gosec) c.rescheduleScan(scan) *pkg/securityscan/jobHandler.go:73:5: G104: Errors unhandled (gosec) c.purgeOldClusterScanReports(scan)
1 parent 2e2cdb9 commit d1da33d

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

pkg/securityscan/jobHandler.go

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,14 @@ func (c *Controller) handleJobs(ctx context.Context) error {
6969
c.setClusterScanStatusDisplay(scan)
7070

7171
if scan.Spec.ScheduledScanConfig != nil && scan.Spec.ScheduledScanConfig.CronSchedule != "" {
72-
c.rescheduleScan(scan)
73-
c.purgeOldClusterScanReports(scan)
72+
err := c.rescheduleScan(scan)
73+
if err != nil {
74+
return obj, fmt.Errorf("error rescheduling scan: %w", err)
75+
}
76+
err = c.purgeOldClusterScanReports(scan)
77+
if err != nil {
78+
return obj, fmt.Errorf("error purging old ClusterScanReports: %w", err)
79+
}
7480
}
7581
err := c.deleteJob(jobs, obj, metav1.DeletePropagationBackground)
7682
if err != nil {

0 commit comments

Comments
 (0)