#174: Confirmation: Some pending topic on Image overview page regarding reports and global filters and more bug fixes

Author: xingzhang-suse

pkg/sbombastic-image-vulnerability-scanner/components/ImageDetails.vue

@@ -98,7 +98,7 @@
         </div>
         <div class="info-item">
           <span class="label">{{ t('imageScanner.imageDetails.imageId') }}:</span>
-          <span class="value">{{ imageDetails.imageId || 'sha256:abc123...' }}</span>
+          <span class="value">{{ imageDetails.imageId?.split(':')[1] || 'Unknown' }}</span>
         </div>
         <div class="info-item">
           <span class="label">{{ t('imageScanner.imageDetails.layers') }}:</span>
@@ -541,21 +541,22 @@ export default {
 
       // Transform the vulnerability data to match the expected format
       return vulnerabilities.map((vuln, index) => ({
-        id:             `${ vuln.cve }-${ vuln.packageName }-${ index }`, // Create unique ID
-        cveId:          vuln.cve,
-        score:          vuln.cvss?.nvd?.v3score ? `${ vuln.cvss.nvd.v3score } (CVSS v3)` : vuln.cvss?.redhat?.v3score ? `${ vuln.cvss.redhat.v3score } (CVSS v3)` : vuln.cvss?.ghsa?.v3score ? `${ vuln.cvss.ghsa.v3score } (CVSS v3)` : '',
-        package:        vuln.packageName,
-        packageVersion: vuln.installedVersion,
-        packagePath:    vuln.purl || vuln.diffID, // Use purl if available, fallback to diffID
-        fixAvailable:   vuln.fixedVersions && vuln.fixedVersions.length > 0,
-        fixVersion:     vuln.fixedVersions ? vuln.fixedVersions.join(', ') : '',
-        severity:       vuln.severity?.toLowerCase() || this.t('imageScanner.general.unknown'),
-        exploitability: vuln.suppressed ? 'suppressed' : 'affected',
-        description:    vuln.description,
-        title:          vuln.title,
-        references:     vuln.references || [],
+        id:               `${ vuln.cve }-${ vuln.packageName }-${ index }`, // Create unique ID
+        cveId:            vuln.cve,
+        score:            vuln.cvss?.nvd?.v3score ? `${ vuln.cvss.nvd.v3score } (CVSS v3)` : vuln.cvss?.redhat?.v3score ? `${ vuln.cvss.redhat.v3score } (CVSS v3)` : vuln.cvss?.ghsa?.v3score ? `${ vuln.cvss.ghsa.v3score } (CVSS v3)` : '',
+        package:          vuln.packageName,
+        packageVersion:   vuln.installedVersion,
+        packagePath:      vuln.purl || vuln.diffID, // Use purl if available, fallback to diffID
+        fixAvailable:     vuln.fixedVersions && vuln.fixedVersions.length > 0,
+        fixVersion:       vuln.fixedVersions ? vuln.fixedVersions.join(', ') : '',
+        severity:         vuln.severity?.toLowerCase() || this.t('imageScanner.general.unknown'),
+        exploitability:   vuln.suppressed ? 'Suppressed' : 'Affected',
+        description:      vuln.description,
+        title:            vuln.title,
+        references:       vuln.references || [],
         // Add diffID for layer grouping
-        diffID:         vuln.diffID
+        diffID:           vuln.diffID,
+        installedVersion: vuln.installedVersion
       }));
     },
 
@@ -909,7 +910,7 @@ export default {
         // Generate CSV from vulnerability report data
         const csvData = this.generateCSVFromVulnerabilityReport();
 
-        this.downloadCSV(csvData, `${ this.imageName }-image-detail-report.csv`);
+        this.downloadCSV(csvData, `${ this.imageName }-image-detail-report_${ day(new Date().getTime()).format('MMDDYYYY_HHmmss') }.csv`);
 
         this.$store.dispatch('growl/success', {
           title:   'Success',
@@ -1162,7 +1163,7 @@ export default {
         // Generate CSV from vulnerability report data
         const csvData = this.generateCSVFromVulnerabilityReport();
 
-        this.downloadCSV(csvData, `${ this.imageName }-full-report.csv`);
+        this.downloadCSV(csvData, `${ this.imageName }-full-report_${ day(new Date().getTime()).format('MMDDYYYY_HHmmss') }.csv`);
 
         this.$store.dispatch('growl/success', {
           title:   'Success',
@@ -1190,7 +1191,7 @@ export default {
         // Generate CSV from filtered vulnerability data
         const csvData = this.generateCSVFromFilteredVulnerabilities();
 
-        this.downloadCSV(csvData, `${ this.imageName }-custom-report.csv`);
+        this.downloadCSV(csvData, `${ this.imageName }-custom-report_${ day(new Date().getTime()).format('MMDDYYYY_HHmmss') }.csv`);
 
         this.$store.dispatch('growl/success', {
           title:   'Success',
@@ -1207,15 +1208,15 @@ export default {
     generateCSVFromVulnerabilityReport() {
       const vulnerabilities = this.vulnerabilityDetails;
       const headers = [
-        'CVE ID',
-        'Score',
-        'Package',
-        'Package Version',
-        'Package Path',
-        'Fix Available',
-        'Fix Version',
-        'Severity',
-        'Exploitability'
+        'CVE_ID',
+        'SCORE',
+        'PACKAGE',
+        'FIX AVAILABLE',
+        'SEVERITY',
+        'EXPLOITABILITY',
+        'PACKAGE VERSION',
+        'PACKAGE PATH',
+        'DESCRIPTION',
       ];
 
       const csvRows = [headers.join(',')];
@@ -1225,12 +1226,12 @@ export default {
           `"${ vuln.cveId || '' }"`,
           `"${ vuln.score || '' }"`,
           `"${ vuln.package || '' }"`,
-          `"${ vuln.packageVersion || '' }"`,
-          `"${ vuln.packagePath || '' }"`,
-          `"${ vuln.fixAvailable ? 'Yes' : 'No' }"`,
-          `"${ vuln.fixVersion || '' }"`,
+          `"${ vuln.fixVersion }"`,
           `"${ vuln.severity || '' }"`,
-          `"${ vuln.exploitability || '' }"`
+          `"${ vuln.exploitability || '' }"`,
+          `"${ vuln.installedVersion || '' }"`,
+          `"${ vuln.packagePath || '' }"`,
+          `"${ vuln.description.replace(/[\r\n]+/g, ' ') }"`,
         ];
 
         csvRows.push(row.join(','));
@@ -1244,15 +1245,15 @@ export default {
       const vulnerabilities = this.selectedVulnerabilities && this.selectedVulnerabilities.length > 0 ? this.selectedVulnerabilities : this.filteredVulnerabilities;
 
       const headers = [
-        'CVE ID',
-        'Score',
-        'Package',
-        'Package Version',
-        'Package Path',
-        'Fix Available',
-        'Fix Version',
-        'Severity',
-        'Exploitability'
+        'CVE_ID',
+        'SCORE',
+        'PACKAGE',
+        'FIX AVAILABLE',
+        'SEVERITY',
+        'EXPLOITABILITY',
+        'PACKAGE VERSION',
+        'PACKAGE PATH',
+        'DESCRIPTION',
       ];
 
       const csvRows = [headers.join(',')];
@@ -1262,12 +1263,12 @@ export default {
           `"${ vuln.cveId || '' }"`,
           `"${ vuln.score || '' }"`,
           `"${ vuln.package || '' }"`,
-          `"${ vuln.packageVersion || '' }"`,
-          `"${ vuln.packagePath || '' }"`,
-          `"${ vuln.fixAvailable ? 'Yes' : 'No' }"`,
-          `"${ vuln.fixVersion || '' }"`,
+          `"${ vuln.fixVersion }"`,
           `"${ vuln.severity || '' }"`,
-          `"${ vuln.exploitability || '' }"`
+          `"${ vuln.exploitability || '' }"`,
+          `"${ vuln.installedVersion || '' }"`,
+          `"${ vuln.packagePath || '' }"`,
+          `"${ vuln.description.replace(/[\r\n]+/g, ' ') }"`,
         ];
 
         csvRows.push(row.join(','));

pkg/sbombastic-image-vulnerability-scanner/config/table-headers.ts

@@ -152,8 +152,9 @@ export const IMAGE_LIST_TABLE = [
   {
     name: "name",
     labelKey: "imageScanner.images.listTable.headers.imageName",
+    value: "imageReference",
     formatter: "ImageNameCell",
-    sort: "name",
+    sort: "imageReference",
   },
   {
     name: "cves",
@@ -181,7 +182,12 @@ export const IMAGE_LIST_TABLE = [
     labelKey: "imageScanner.images.listTable.headers.registry",
     value: "imageMetadata.registry",
     formatter: "RegistryCellLink",
-    sort: "imageMetadata.registry",
+    sort: ['metadata.namespace','imageMetadata.registry'],
+  },{
+    name: "repository",
+    labelKey: "imageScanner.images.listTable.headers.repository",
+    value: "imageMetadata.repository",
+    sort: 'imageMetadata.repository',
   },
   {
     name: "platform",