Skip to content

HTTP Client wrong encoding #566

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

Merged
merged 2 commits into from
Jun 10, 2021
Merged

Conversation

amortJ
Copy link
Contributor

@amortJ amortJ commented Jun 7, 2021

Description
The Apache Http client uses the ISO-8859-1 encoding for request bodies. This is not compatible with the rest endpoint and the encoding declared in the header of the request (UTF-8).

Fixed issue: #565

The Apache Http client uses the ISO-8859-1 encoding for request bodies. This is not compatible with the rest endpoint and the encoding declared in the header of the request (UTF-8).

Adyen#565
The Apache Http client uses the ISO-8859-1 encoding for request bodies. This is not compatible with the rest endpoint and the encoding declared in the header of the request (UTF-8).

Adyen#565
@coveralls
Copy link

Coverage Status

Coverage increased (+0.002%) to 29.862% when pulling 317d3da on amortJ:bugfix/http_client_encoding into 82a09bc on Adyen:develop.

@amortJ
Copy link
Contributor Author

amortJ commented Jun 9, 2021

Hi @Aleffio,
sorry for contacting you directly once more.
This issue is blocking us from upgrading to the latest version of the library.
Would be great if you could review and release these changes soon.

Thanks,
Jan

@Aleffio
Copy link
Contributor

Aleffio commented Jun 10, 2021

Hi @amortJ ,
Thank you for opening this pull request and fixing the bug!
We will merge this and release a new version soon.
Regards,
Alessio
Adyen

@Aleffio Aleffio merged commit 11fe674 into Adyen:develop Jun 10, 2021
@amortJ
Copy link
Contributor Author

amortJ commented Jun 16, 2021

Hi @Aleffio,

i hate to be pushy but we really need these changes released.

Thanks.
Jan

@Aleffio Aleffio mentioned this pull request Jun 18, 2021
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.

5 participants