Skip to content

release: Ignore false positive CVE#38293

Draft
radeksimko wants to merge 1 commit intomainfrom
radek/ignore-false-positive-cve
Draft

release: Ignore false positive CVE#38293
radeksimko wants to merge 1 commit intomainfrom
radek/ignore-false-positive-cve

Conversation

@radeksimko
Copy link
Member

This is to unblock release by suppressing scanner noise on a CVE which does not impact Terraform since go-plugin or Terraform do not perform auth on that layer.

Whilst there is a patch in #38292 gRPC could generally be considered a relatively critical dependency and bumping it increases risks of breaking things purely to satisfy a scanner.

@radeksimko radeksimko added 1.14-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged 1.15-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged no-changelog-needed Add this to your PR if the change does not require a changelog entry labels Mar 19, 2026
Copy link
Member

@dbanck dbanck left a comment

Choose a reason for hiding this comment

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

TIL, I didn't know we have this option 💯

@dbanck
Copy link
Member

dbanck commented Mar 19, 2026

One thought: can scans of the compiled binary still pick this up? In that case, we may still see reports

@radeksimko
Copy link
Member Author

One thought: can scans of the compiled binary still pick this up?

I'd hope not, at least that is why the settings is nested under binary block - I'd expect it to suppress scanning of compiled binaries. I'm still waiting for some confirmation from the ProdSec team though before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.14-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged 1.15-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged no-changelog-needed Add this to your PR if the change does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants