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