Add support for Apache HttpClient 5.0

[mdeinum]

Current Apache Http Client 4.x is supported however there is also Apache Http Client 5.0 (and I believe 5.1 is in the making), which contains some breaking changes. Might be worthwhile to add support for Http Client 5.0 next to the 4.5.

Or add some documentation on how to configure the Apache Http Client using the ClientHttpRequestFactory abstraction in Spring itself to configure the Apache Http Client, this would save adding additional support into this and use the abstraction from Spring itself instead.

[zielarz25]

+1 It becomes even more important now, when spring-mvc 6.x requires Apache Http Client 5.1+, but spring-ws still needs httpclient 4.x, resulting in some projects having two similar dependencies, potentially conflicting ones

[mdeinum]

You don't need to use the http client v4, as mentioned you can also use the ClientHttpRequestFactory (the infrastructure used by the RestTemplate) and you can use any supported version. Maybe that should become the default instead of have different implementation for different clients.

[gregturn]

Apparently, there are some official guides for migrating: https://hc.apache.org/httpcomponents-client-5.2.x/migration-guide/index.html

I need to study them to ensure we are properly aligning with their expectations.

[gregturn]

@artemptushkin I invite you to track this ticket on our progress towards HttpClient 5 support.

[gregturn]

As a reminder, this ticket is aimed at ADDING support for HttpClient 5, not breaking current users presently on HttpClient 4.x.

I will further check what Spring Framework is doing to see whether or not adding such support would dictate deprecating HttpClient 4 support or not.

[daromnik]

Hello, is there any news on this issue ?

[gregturn]

I did a first cut implementing it, and the code compiled, but the test cases all broke with some mysterious "Stream already closed" exception that I have yet to debug. I'm not sure if it's how I'm using the code, or if it is simply a false expectation in the test cases that needs to be shook out.

[Facedancer1984]

Hello.
Are there any updates on this issue?