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.

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

GetAddress not present in httpcore < 4.3 #7674

Closed
anuragagarwal561994 opened this issue Jan 30, 2023 · 4 comments
Closed

GetAddress not present in httpcore < 4.3 #7674

anuragagarwal561994 opened this issue Jan 30, 2023 · 4 comments
Labels
contribution welcome Request makes sense, maintainers probably won't have time, contribution would be welcome enhancement New feature or request

Comments

@anuragagarwal561994
Copy link
Contributor

So we have used target.getAddress() here but the method was added in http core v4.3

If I see on maven central with apache httpasyncclient v4.1 https://mvnrepository.com/artifact/org.apache.httpcomponents/httpasyncclient/4.1

The default version that comes along for httpcore is 4.4.x but that doesn't mean that the user doesn't have a way to override this.

Hence someone using httpasyncclient 4.1.x with < 4.4.x version of httpcore, things will break.

Originally posted by @anuragagarwal561994 in #6503 (comment)

@anuragagarwal561994
Copy link
Contributor Author

Ideally imho a lot of things changed between 4.0 and 4.3 version of httpcore.

I believe that a separate implementation of the same can be built if it adds value

@mateuszrzeszutek mateuszrzeszutek added enhancement New feature or request contribution welcome Request makes sense, maintainers probably won't have time, contribution would be welcome labels Jan 31, 2023
@laurit
Copy link
Contributor

laurit commented Feb 7, 2023

@anuragagarwal561994 do you have an application that uses httpasyncclient 4.1.x with httpcore < 4.4.x? Is this issue created out of necessity to support such configurations or does it only imply that there is a theoretical possibility to mix these components in such a way? As far as I can tell httpasyncclient 4.1 uses classes that were added in httpcore 4.4, such as https://github.com/apache/httpcomponents-client/blob/rel/v4.4/httpclient/src/main/java/org/apache/http/conn/util/PublicSuffixMatcherLoader.java, which makes me question the viability of using httpcore < 4.4.x? with httpasyncclient 4.1.x.

@laurit laurit added the needs author feedback Waiting for additional feedback from the author label Feb 7, 2023
@anuragagarwal561994
Copy link
Contributor Author

I will check this once @laurit by creating a sample application. So generally we can't rely on developers to update 2 separate dependencies by checking their compatibility generally we update the ones where we want to use the newer functions of a library. If a application is using http client and http async client where in httpclient they have specified httpcore < 4.4 then issue can come up. But I will check if httpasyncclient even works with < 4.4 because in that case the developer will be forced to upgrade httpcore even when they have mentioned in their pom.xml file.

@github-actions github-actions bot removed the needs author feedback Waiting for additional feedback from the author label Feb 7, 2023
@anuragagarwal561994
Copy link
Contributor Author

anuragagarwal561994 commented Feb 28, 2023

@laurit @trask

Release 4.0 Beta 4


The 4.0 BETA4 release delivers significant performance improvements in request execution,
especially for short HTTP messages, and also re-aligns programming interfaces used by the library
with HttpCore 4.3 and HttpClient 4.3 APIs. Configuration and preference APIs of HttpAsyncClient
are now consistent with those used by HttpClient 4.3.

Users of previous releases of HttpAsyncClient are advised to upgrade.

Quoting the release notes we can confirm that httpasyncclient is compatible post httcore 4.3

Hence we can close this issue

@laurit laurit closed this as completed Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution welcome Request makes sense, maintainers probably won't have time, contribution would be welcome enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants