fix: Fix for using charset from response headers instead of default UTF-8
Description
The REST-API response content are by default parsed in UTF-8 irrespective of what the response content-type header charset is sent.
Fixes #
Checks for any charset mentioned in response headers, if mentioned, it used the charset otherwise defaults to UTF-8
Type of change
- New feature (non-breaking change which adds functionality)
How Has This Been Tested?
- Manual
Checklist:
Dev activity
- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
- [ ] PR is being merged under a feature flag
Welcome to the Appsmith community! Thank you for your first pull request and making this project better. 🤗 Please make sure that you raise a review request so your code change does not go unnoticed.
@balachandarsv is attempting to deploy a commit to the Appsmith Team on Vercel.
A member of the Team first needs to authorize it.
This PR has not seen activitiy for a while. It will be closed in 7 days unless further activity is detected.
This PR has been closed because of inactivity.
@balachandarsv Thank you for opening this PR. Apologies that it slipped through the cracks. I've reopened the PR and we'll work to merge this in. In the meantime, can I request you to please open a Github issue for this?
Hi @balachandarsv , please mention if you are still working on this PR.