feat: Add jsonStrings configuration

[rubenmorim]

Added configuration to support jsonStrings.
Check #1072 for more details about this feat

[wlarch]

Seems perfectly legit to me. 💪

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.32%. Comparing base (e536b41) to head (fac1a87).
Report is 8 commits behind head on master.

Additional details and impacted files

@@ Coverage Diff @@ ## master #2642 +/- ## ======================================= Coverage 90.32% 90.32% ======================================= Files 71 71 Lines 15727 15729 +2 Branches 1339 1342 +3 ======================================= + Hits 14206 14208 +2  Misses 1521 1521 

Flag	Coverage Δ
compression-0	90.32% <100.00%> (+<0.01%)	⬆️
compression-1	90.32% <100.00%> (+<0.01%)	⬆️
tls-0	89.85% <100.00%> (+<0.01%)	⬆️
tls-1	90.15% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[wellwelwel]

Hey, @rubenmorim. Thanks 🚀

---

I believe the changes also fit the binary parser:

node-mysql2/lib/parsers/binary_parser.js

Line 62 in e536b41

return 'JSON.parse(packet.readLengthCodedString("utf8"));';

---

For typings, we can include the option jsonStrings to node-mysql2/typings/mysql/lib/Connection.d.ts:

{ jsonStrings?: boolean; }

• If add this feature to query level, include it in node-mysql2/typings/mysql/lib/protocol/sequences/Query.d.ts.

---

Also, we could have a basic test for execute and query. What do you think? 🙋🏻‍♂️
For example, something like:

SELECT CAST('{"test": true}' AS JSON) AS json_result;

[sidorares]

lgtm, in addition to @wellwelwel comments - maybe also add a bit of documentation somewhere in https://github.com/sidorares/node-mysql2/tree/master/website/docs/documentation

[wellwelwel]

The added documentation also closes:

• JSON incompatibility with node-mysql #2181

[wellwelwel]

Thanks again, @rubenmorim. I'll include the tests in a separate PR (#2720).

[rubenmorim]

Thanks again, @rubenmorim. I'll include the tests in a separate PR (#2720).

Great ! thank you guys @sidorares @wellwelwel