Skip to content
This repository has been archived by the owner on May 11, 2020. It is now read-only.

Please don't assume client application encoding #10

Open
cmanley opened this issue Jul 20, 2018 · 0 comments
Open

Please don't assume client application encoding #10

cmanley opened this issue Jul 20, 2018 · 0 comments

Comments

@cmanley
Copy link

cmanley commented Jul 20, 2018

The current version assumes the method input and output is UTF-8. That is not necessarily the case (it could be cp1252 for example). It's best to detect application encoding using mb_internal_encoding() and then transcode input and output, especially before/after calls to json_ecode() and json_decode().

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant