Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

False positive about a vulnerable action version number #245

Open
johnbillion opened this issue Dec 10, 2024 · 1 comment
Open

False positive about a vulnerable action version number #245

johnbillion opened this issue Dec 10, 2024 · 1 comment

Comments

@johnbillion
Copy link

Describe the bug

I'm getting the following error for a workflow that uses actions/download-artifact@v4:

The workflow or action depends on a GitHub Action with known vulnerabilities.

Reading GHSA-cxww-7g56-2vh6 for actions/download-artifact, the description says:

Alternatively use 'v4' tag which points to the latest and secure version.

To Reproduce

  1. Create a workflow file that uses actions/download-artifact@v4
  2. Run Poutine on the file
  3. Observe the above error message

Expected behavior

Using @v4 should be considered "safe" as far as this rule is concerned. Whether or not it should use a pinned sha hash is a separate concern.

Screenshots

@fproulx-boostsecurity
Copy link
Contributor

@johnbillion thanks for the report. Looks like in order to fix this we will need to make improvements to the matching logic. As when we see “v4” we currently expand it to “v4.0.0” which matches in the vulnerable range. I think we will have to have an exclusion mechanism to allow those exact match on mutable tags.

The structure will support something like “non_vulnerable_exact_versions” https://github.com/boostsecurityio/poutine/blob/main/opa/rego/external/osv.rego#L17

We don’t have so much bandwidth to fix in next 2 weeks. If you’d like to contribute a PR we’ll be happy to review. Otherwise hopefully we can get that before Christmas.

Cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants