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

build(deps): Update mysql and postgres drivers of the docker image #7429

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

lloyd-theophilus
Copy link
Contributor

Description of Change

Related issues

Compatibly issue

Have test cases been added to cover the new functionality?

No test cases need as I only change the driver versions to the latest
yes/no

@lloyd-theophilus lloyd-theophilus changed the title Update mysql and postgres driver version fix/Update mysql and postgres driver version Feb 18, 2025
@aikebah aikebah changed the title fix/Update mysql and postgres driver version build(deps): Update mysql and postgres drivers of the docker image Feb 18, 2025
Copy link
Collaborator

@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 b41a22f into dependency-check:main Feb 19, 2025
8 of 10 checks passed
@jeremylong
Copy link
Collaborator

@aikebah
Copy link
Collaborator

aikebah commented Feb 19, 2025

@jeremylong Connector-J is for current versions at
https://dev.mysql.com/get/Downloads/Connector-J/mysql-connector-j-<version>.tar.gz

instead of the previous

https://dev.mysql.com/get/Downloads/Connector-J/mysql-connector-java-<version>.tar.gz

For Postgresql I'm not sure whether the space after the equal-sign would be included in the URL, if it is retained that would also break.

But first some sleep and then another day in the office for me. If you don't get to it I might prepare a proper PR tomorrow evening.

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.

3 participants