Skip to content

Conversation

@Linda-pan
Copy link
Contributor

Fix bug for FeignException cannot get the correct charset(#1325)

Copy link
Member

@kdavisk6 kdavisk6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Linda-pan Please add a unit test for this use case.

@kdavisk6 kdavisk6 added the feedback provided Feedback has been provided to the author label Dec 29, 2020
@kdavisk6
Copy link
Member

@Linda-pan Your latest changes are not formatted correctly, please fun mvn install locally to format the files, commit the changes and push the updates to fix the build

@Linda-pan Linda-pan requested a review from kdavisk6 January 14, 2021 06:35
Copy link
Contributor Author

@Linda-pan Linda-pan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix bug for FeignException cannot get the correct charset and add some tests

@kdavisk6 kdavisk6 added ready to merge Will be merged if no other member ask for changes and removed feedback provided Feedback has been provided to the author labels Mar 7, 2021
@kdavisk6 kdavisk6 merged commit c8fbb85 into OpenFeign:master Mar 7, 2021
velo pushed a commit that referenced this pull request Oct 7, 2024
)

* Fix bug for FeignException cannot get the correct charset (#1325)

* Add test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Add more test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Format test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Fix bug for FeignException cannot get the correct charset (#1325)

* Add test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Add more test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Format test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Add test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Add more test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Format test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Correcting License Headers for 2021

Co-authored-by: Kevin Davis <kdavisk6@gmail.com>
velo pushed a commit that referenced this pull request Oct 8, 2024
)

* Fix bug for FeignException cannot get the correct charset (#1325)

* Add test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Add more test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Format test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Fix bug for FeignException cannot get the correct charset (#1325)

* Add test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Add more test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Format test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Add test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Add more test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Format test for (Fix bug for FeignException cannot get the correct charset) (#1325)

* Correcting License Headers for 2021

Co-authored-by: Kevin Davis <kdavisk6@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Will be merged if no other member ask for changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants