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

feat: add mysql8 support, update to mysql2 client, drop nodejs v14 and mysql5.7 support [BREAKING CHANGE] #484

Merged
merged 1 commit into from
Jun 30, 2023

Conversation

samarpanB
Copy link
Collaborator

@samarpanB samarpanB commented Jun 18, 2023

Add mysql8 support and change to mysql2 client library

BREAKING CHANGE
update mysql client to mysql2 client as that support new authentication protocol of mysql 8
drop mysql5.7 support
drop nodejs v14 support

MySQL 5.7 EOL is in October 2023 - https://endoflife.software/applications/databases/mysql. So we will only keep it for security updates in separate branch.
@achrinza @dhmlau any specific approach to be taken for that, please let me know.

Fixes #477 , #434, #468

Checklist

  • DCO (Developer Certificate of Origin) signed in all commits
  • npm test passes on your machine
  • New tests added or existing tests modified to cover all changes
  • Code conforms with the style guide
  • Commit messages are following our guidelines

@dhmlau
Copy link
Member

dhmlau commented Jun 19, 2023

@samarpanB, since you're working on a breaking change, could you please also drop the support for Node.js 14 which reaches end of life already? Thanks!

@samarpanB samarpanB marked this pull request as ready for review June 25, 2023 19:52
@samarpanB samarpanB force-pushed the gh-477 branch 2 times, most recently from 620581e to 45d6f0e Compare June 25, 2023 20:07
@samarpanB samarpanB changed the title feat: add mysql8 support and update to mysql2 client [BREAKING CHANGE] feat: add mysql8 support, update to mysql2 client, drop nodejs v14 and mysql5.7 support [BREAKING CHANGE] Jun 25, 2023
@samarpanB samarpanB force-pushed the gh-477 branch 9 times, most recently from 29f0641 to 9411e86 Compare June 26, 2023 12:59
Copy link
Member

@dhmlau dhmlau left a comment

Choose a reason for hiding this comment

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

LGTM

… and mysql5.7 support

BREAKING CHANGE
drop mysql5.7 support

Signed-off-by: Samarpan Bhattacharya <[email protected]>
@samarpanB samarpanB merged commit 07173b6 into master Jun 30, 2023
6 checks passed
@samarpanB samarpanB deleted the gh-477 branch June 30, 2023 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for nodejs v18 and mysql v8
3 participants