Use mysql2 lib to be able to use MySQL 8

[atog]

No description provided.

[fengzai6]

I need it too

[fengzai6]

sometime use find function will throw error：AdapterError: Unexpected error from database adapter: "[object Object]" is not valid JSON

[Harm-Nullix]

Why is this not merged by 2024?
https://endoflife.date/mysql

[Harm-Nullix]

@mikermcneil ,
@eashaw , any light on this?

[MarijnMensinga]

@fengzai6 and @Harm-Nullix I found the issue. If a model property has type json and the database fieldType is also json mysql2 returns the data as parsed (json) object instead of the json string. Therefore a small change is needed in sails-mysql as well. Since this ticket/library seems dormant I've created a fork with the fixed in it. If you want you could add
"sails-mysql": "https://gitlab.com/newtee-public/balderdashy-sails-mysql.git" to your package.json and you'll be able to use mysql 8 with your sails backend. It also packs the fixed dependency from this PR: https://gitlab.com/newtee-public/sailshq-machinepack-mysql2

[fengzai6]

@fengzai6 and @Harm-Nullix I found the issue. If a model property has type json and the database fieldType is also json mysql2 returns the data as parsed (json) object instead of the json string. Therefore a small change is needed in sails-mysql as well. Since this ticket/library seems dormant I've created a fork with the fixed in it. If you want you could add "sails-mysql": "https://gitlab.com/newtee-public/balderdashy-sails-mysql.git" to your package.json and you'll be able to use mysql 8 with your sails backend. It also packs the fixed dependency from this PR: https://gitlab.com/newtee-public/sailshq-machinepack-mysql2

OK，I will try it, thx!

[maheshwarishivam]

@eashaw Any update on this. When can we have MySQL 8 compatibility?

[kei-katahira]

You might consider setting the jsonStrings option in MySQL2 to true.
This will ensure that JSON data retrieved from MySQL is consistently returned as strings and avoid parsing problems.

https://sidorares.github.io/node-mysql2/docs/documentation#known-incompatibilities-with-node-mysql