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

Mysql-connector-java upgrade #15422

Closed
mustajibmk opened this issue Nov 23, 2023 · 2 comments
Closed

Mysql-connector-java upgrade #15422

mustajibmk opened this issue Nov 23, 2023 · 2 comments

Comments

@mustajibmk
Copy link

I am working on upgrading mysql-connector-java to version 8.0.28 - PR. This is needed to resolve security vulnerabilities. I creating this issue to discuss the obstacles that I am facing during the process. Please let me know if this is not the right place.

With the upgrade the com.mysql.jdbc.NonRegisteringDriver#parseURL method used in org.apache.druid.utils.ConnectionUriUtils is no longer available. So, I have resorted to instantiating com.mysql.cj.conf.ConnectionUrlParser with the connection string and then used the "getProperties" method to fetch the properties. This method is still not a like-to-like replacement as we do not get the response back in the earlier format.

Is com.mysql.cj.conf.ConnectionUrlParser the right way to proceed or should I write a custom parser based on the logic previous used or is there something else I should look at? Appreciate your help!

@mustajibmk mustajibmk changed the title Mysql-connectorjava upgrade Mysql-connector-java upgrade Nov 23, 2023
@liyaka
Copy link

liyaka commented Aug 1, 2024

@mustajibmk
any update on this?

@FrankChen021
Copy link
Member

The work has been done. it has been upgraded to 8.2 via #16024

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

No branches or pull requests

3 participants