Skip to content

#1166 add a column containing image digest in generated reports for containers#1193

Merged
xingzhang-suse merged 2 commits into
neuvector:mainfrom
rushk014:main
May 6, 2026
Merged

#1166 add a column containing image digest in generated reports for containers#1193
xingzhang-suse merged 2 commits into
neuvector:mainfrom
rushk014:main

Conversation

@rushk014
Copy link
Copy Markdown
Contributor

@rushk014 rushk014 commented May 4, 2026

Description

Fix #1166, Fix #1192

Adds a dedicated digest column to the container vulnerability scan report CSV export, and fixes image name/tag parsing for image references that include registry ports or digest-only refs.

Test

  1. Navigate to Assets → Containers
  2. Download Full Report CSV
  3. Verify the header row contains digest
  4. Verify the digest column is populated from workload_image_digest in the API response (including empty case)
  5. (for [UI] Image name and tag incorrectly parsed in full container scan report when registry contains a port #1192) Verify image_name and tags are correctly split for images with registry ports (e.g. 127.0.0.1:31999/repo/img:v1)

Split on the last colon after the final slash to avoid treating
registry ports as part of the tag. Strip
@digest so refs don't pollute the tag field.

Fixes neuvector#1192
@rushk014 rushk014 requested a review from a team as a code owner May 4, 2026 22:51
@rushk014 rushk014 requested a review from jchan-suse May 4, 2026 22:51
@rushk014 rushk014 marked this pull request as draft May 4, 2026 23:28
…ed reports for containers (UI)

The backend POST /v1/scan/workloads/scan_report now returns
workload_image_digest (string) per CVE record containing the resolved
sha256 digest, or empty if the enforcer cannot determine it (e.g.
conflicting digests in Docker runtime). Surface this as a dedicated
'digest' column after 'tags' in the exported CSV.
@rushk014 rushk014 marked this pull request as ready for review May 6, 2026 22:28
Copy link
Copy Markdown
Collaborator

@xingzhang-suse xingzhang-suse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@xingzhang-suse xingzhang-suse merged commit e237d8e into neuvector:main May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants