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.

Sign up for GitHub

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

Jump to bottom

Change the http client implementation that nacos resttemplate depends on from apache to JDk #3317

Closed
Maijh97 opened this issue Jul 14, 2020 · 0 comments · Fixed by #3322
Closed

Change the http client implementation that nacos resttemplate depends on from apache to JDk #3317

Maijh97 opened this issue Jul 14, 2020 · 0 comments · Fixed by #3322
Assignees
@Maijh97

Comments

@Maijh97
Copy link
Collaborator

Maijh97 commented Jul 14, 2020 •
edited
Loading

Issue Description

Type: bug report or feature request

Describe what happened (or what feature you want)

refer: #2858
To avoid unknown exceptions caused by changes to components that nacos-client depends on due to the unified http client task, the HTTP client implementation that nacos resttemplate depends on was changed from apache to JDK.

Describe what you expected to happen

How to reproduce it (as minimally and precisely as possible)

Tell us your environment

Anything else we need to know?
@Maijh97 Maijh97 mentioned this issue Jul 14, 2020
Merged
5 tasks
KomachiSion pushed a commit that referenced this issue Jul 14, 2020
@Maijh97
[ISSUE##3317]Change the http client implementation that nacos resttem…
132adb4 
…plate depends on from apache to JDk (#3322)

* fix:#3317 change the http client implementation that nacos resttemplate depends on from apache to JDk

* Use the IoUtils.closeQuietly() method to close the InputStream

* change HttpClientBeanHolder logger output object
@KomachiSion KomachiSion mentioned this issue Aug 4, 2020
Merged
5 tasks
@huhu-sky huhu-sky mentioned this issue May 15, 2021
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Maijh97 Maijh97

Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@Maijh97