Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug fix #124: Unicode support for execute-request #128

Merged
merged 6 commits into from
Jul 19, 2022

Conversation

symphony-soufiane
Copy link
Contributor

@symphony-soufiane symphony-soufiane commented Jul 18, 2022

#124: Unicode support for execute-request.
When a body is sent with multipart-form-data content type, we were using the default content type when adding the multipart string to the actual MultipartEntity object. This default content type is plain/text with ISO_8859_1 encoding charset. The fix was to enforce using StandarCharsets.UTF_8

When a body is sent with multipart-form-dara content type, we were using the default content type when adding the multipart string to the actual MultipartEntity object. This default content type is plain/text with ISO_8859_1 encoding charset. The fix was to enforce using StandarCharsets.UTF_8.
@symphony-soufiane symphony-soufiane requested a review from a team July 18, 2022 12:45
@symphony-soufiane symphony-soufiane linked an issue Jul 18, 2022 that may be closed by this pull request
@symphony-soufiane symphony-soufiane merged commit 1cb59ec into finos:master Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unicode Support for execute-request
2 participants