-
Notifications
You must be signed in to change notification settings - Fork 33
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
Upgrade httpclient to 5.x #89
Comments
Are the artifacts available somewhere? Maven is only showing 4.x. https://mvnrepository.com/artifact/org.apache.httpcomponents/httpclient Edit: Nevermind, found them at a new namespace: https://mvnrepository.com/artifact/org.apache.httpcomponents.client5 Will probably be a bit of work to upgrade, since a bunch of imports will have changed. |
I don’t know why the major version was included in the name (perhaps to
facilitate the coexistence of 4.x and 5.x?), but here it is:
https://mvnrepository.com/artifact/org.apache.httpcomponents.client5/httpclient5/5.0.3
…On Fri, 30 Oct 2020 at 19:00, Chris Heald ***@***.***> wrote:
Are the artifacts available somewhere? Maven is only showing 4.x.
https://mvnrepository.com/artifact/org.apache.httpcomponents/httpclient
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#89 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAHYQP3TYLK6OREHPSIQR3SNMEMLANCNFSM4TE72LJA>
.
|
It is also recommended to update due to vulnerabilities |
I started in on this at one point, but there were deprecations, primarily around HttpEntity, that I never finished resolving. A pull request would be very welcome here. |
It seems that new fixes and enhancements are no longer added to httpclient 4.x so it may be time to adopt 5.x in this project as well.
An example of the need for this is support of 308 redirects, which has only been added to 5.0.x https://github.com/apache/httpcomponents-client/commit/2b6f7e5/
The text was updated successfully, but these errors were encountered: