ignore the UnicodeDecodeError when some bytes cannot be decoded with … #583
Conversation
…the specified charset use env variable MYC_USE_DECODE_ERR_REP_MODE to switch on/off, if MYC_USE_DECODE_ERR_REP_MODE is 1, then use replace mode, or NULL mode
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| } else { | ||
| //fprintf(stderr, "decoding with %s\n", encoding); | ||
| return PyUnicode_Decode(rowitem, length, encoding, NULL); | ||
| return PyUnicode_Decode(rowitem, length, encoding, decode_err_mode); |
Check notice
Code scanning / CodeQL
Commented-out code
There was a problem hiding this comment.
it's existed before my modifcation.
| if (encoding == utf8) { | ||
| //fprintf(stderr, "decoding with utf8!\n"); | ||
| return PyUnicode_DecodeUTF8(rowitem, length, NULL); | ||
| return PyUnicode_DecodeUTF8(rowitem, length, decode_err_mode); |
Check notice
Code scanning / CodeQL
Commented-out code
There was a problem hiding this comment.
it's existed before my modifcation.
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #583 +/- ##
=======================================
Coverage 86.00% 86.00%
=======================================
Files 6 6
Lines 550 550
=======================================
Hits 473 473
Misses 77 77 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
|
I don't maintain such code. |
sometime the function cursor.fetchall() raise exceptions like follower:
this exception is very annoying if you don't fix the records which include the Garbled character.