Fix wrong scanned artifact count when there are multiple report for a…

…n artifact Signed-off-by: stonezdj <daojunz@vmware.com>
goharbor · Jul 21, 2023 · 5b24e59 · 5b24e59
1 parent d4aa9b1
commit 5b24e59
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/pkg/securityhub/dao/security.go b/src/pkg/securityhub/dao/security.go
@@ -48,10 +48,9 @@ order by s.critical_cnt desc, s.high_cnt desc, s.medium_cnt desc, s.low_cnt desc
 limit 5`
 
 	// sql to query the scanned artifact count
-	scannedArtifactCountSQL = `select count(1) 
-           from artifact a 
-      left join scan_report s on a.digest = s.digest 
-          where s.registration_uuid= ? and s.uuid is not null`
+	scannedArtifactCountSQL = `select count(1)
+from artifact
+where exists (select 1 from scan_report s where artifact.digest = artifact.digest and s.registration_uuid = ?) `
 
 	// sql to query the dangerous CVEs
 	dangerousCVESQL = `select vr.*