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

fix(fp): Improve suppression for Glassfish Server false positives (round 2) #7024

Conversation

chadlwilson
Copy link
Contributor

Fixes Issue

Description of Change

Extends suppression from #7016 to also suppress for projects under the org.glassfish group; such as javax.json, jakarta.json etc. it seems the full Glassfish server has always been published under org.glassfish.main, although there are probably artifacts in subgroup for .main which are also false positives, so might require someone with more familiarity with Glassfish server to tweak further.

Have test cases been added to cover the new functionality?

N/A

Extends suppression to also suppress for projects under the `org.glassfish` group; such as `javax.json`, `jakarta.json` etc.

Signed-off-by: Chad Wilson <[email protected]>
Copy link
Owner

@jeremylong jeremylong left a comment

Choose a reason for hiding this comment

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

LGTM

@jeremylong jeremylong merged commit 2865d47 into jeremylong:generatedSuppressions Oct 9, 2024
1 check failed
@chadlwilson chadlwilson deleted the improve-glassfish-suppression branch October 9, 2024 12:22
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants