Skip to content

fix(fp): extend Sentry server CPE suppression to other ecosystems#8524

Open
chadlwilson wants to merge 2 commits into
dependency-check:generatedSuppressionsfrom
chadlwilson:generatedSuppressions-improve-sentry-suppression
Open

fix(fp): extend Sentry server CPE suppression to other ecosystems#8524
chadlwilson wants to merge 2 commits into
dependency-check:generatedSuppressionsfrom
chadlwilson:generatedSuppressions-improve-sentry-suppression

Conversation

@chadlwilson
Copy link
Copy Markdown
Collaborator

@chadlwilson chadlwilson commented May 20, 2026

Description of Change

This is an extension of earlier PR #8058 to suppress for other ecosystems - excluding docker, generic.

There are hundreds of sentry packages across various ecossytems, from maven to composer, cargo, npm, rubygems, go and swift. The self-hosted server is Python (and typescript) based and used to be distributed on the previously excluded Python package, but is now distributed as docker images with pre-compiled Python: https://github.com/getsentry/sentry/blob/master/self-hosted/Dockerfile

This is all a bit of a minefield, but it should be relatively safe to extend the suppression as suggested here, while still matching for folks against the self-hosted instance (even though we don't support discovering docker dependencies right now).

Related issues

Have test cases been added to cover the new functionality?

yes/no

@chadlwilson chadlwilson requested a review from Copilot May 20, 2026 16:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates a hand-curated suppression to reduce false positives for Sentry Server CVEs by refining the notes and the suppression match patterns.

Changes:

  • Expanded the suppression notes with additional issue context and an NVD query link.
  • Modified the packageUrl regex to broaden/adjust which ecosystems are affected by the suppression.
  • Adjusted the cpe regex to match cpe:/a:sentry:sentry with optional suffix.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread generatedSuppressions.xml
Comment thread generatedSuppressions.xml
Comment thread generatedSuppressions.xml
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

Successfully merging this pull request may close these issues.

2 participants