The `ignore-base`, `ignore-unchanged`, `only-fixed`, `only-severities` parameters should add-up together
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 141
- Forks
- 53
- PR merge metrics
- No merged PRs in 30d
Description
Issue
I've been implementing this action to ensure our engineering team is informed if new vulnerabilities are introduced into our Docker image through the development process, particularly in PRs.
We want to avoid overwhelming the team with action comments in the PRs, as this could lead to warning fatigue. If notifications aren't actionable, they will likely be ignored over time.
Expected behavior
To address this, I would expect the ignore-unchanged option to prevent cves, recommendations, or compare information from being displayed when no new vulnerabilities have been introduced compared to the base image.
Additionally, the following options should work together: ignore-base, ignore-unchanged, only-fixed, only-severities.
Here’s what I envision:
with:
command: quickview,cves,recommendations,compare
image: <built-image>
to: <base-image>
ignore-base: true
ignore-unchanged: true
only-fixed: true
only-severities: critical,high
This configuration would only display information in a PR if: there are new fixable high or critical vulnerabilities in the built image. Otherwise no information is displayed in the PR.
Current behavior
Even with all these parameters, comments will be displayed under the above condition.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
The issue does not mention specific files or tests. Start by tracing how the ignore-base, ignore-unchanged, only-fixed, and only-severities parameters are handled for the quickview, cves, recommendations, and compare commands. Done means a pull request comment appears only when new fixable high- or critical-severity vulnerabilities are present.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- docker, github-actions, javascript
- Domain
- ci-cd, security
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100