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

[FP]: Many FPs on mysql-connector-j that are for the mysql server #7057

Closed
chadlwilson opened this issue Oct 17, 2024 · 4 comments
Closed

[FP]: Many FPs on mysql-connector-j that are for the mysql server #7057

chadlwilson opened this issue Oct 17, 2024 · 4 comments
Labels
FP Report maven changes to the maven plugin

Comments

@chadlwilson
Copy link
Contributor

Package URl

pkg:maven/com.mysql/[email protected]

CPE

cpe:2.3:a:mysql:mysql:9.0.0:*:*:*:*:*:*:*

CVE

CVE-2024-21196, CVE-2024-21230, CVE-2024-21238 etc etc

ODC Integration

{"label"=>"Gradle Plugin"}

ODC Version

10.0.4

Description

Historically individual CVE suppressions were used, however the mysql connectors have their own CPEs as below so this is no longer appropriate. Will submit PRs to correct this and reduce ongoing maintenance headache.

cpe:2.3:a:oracle:mysql_connector\/j:9.0.0:*:*:*:*:*:*:* 
cpe:2.3:a:oracle:mysql_connectorsj:9.0.0:*:*:*:*:*:*:* 

https://github.com/jeremylong/DependencyCheck/blob/main/core/src/main/resources/dependencycheck-base-suppression.xml#L3210-L3818

Copy link
Contributor

Maven Coordinates

<dependency>
   <groupId>com.mysql</groupId>
   <artifactId>mysql-connector-j</artifactId>
   <version>9.0.0</version>
</dependency>

Suppression rule:

<suppress base="true">
   <notes><![CDATA[
   FP per issue #7057
   ]]></notes>
   <packageUrl regex="true">^pkg:maven/com\.mysql/mysql-connector-j@.*$</packageUrl>
   <cpe>cpe:/a:mysql:mysql</cpe>
</suppress>

Link to test results: https://github.com/jeremylong/DependencyCheck/actions/runs/11377631854

@chadlwilson
Copy link
Contributor Author

Fixed in #7058 (still to be published on the hosted suppressions at time of writing)

@aikebah
Copy link
Collaborator

aikebah commented Oct 20, 2024

published now with the help of another FP Report workflow

@chadlwilson
Copy link
Contributor Author

Cool, will submit cleanup PR to master shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FP Report maven changes to the maven plugin
Projects
None yet
Development

No branches or pull requests

2 participants