Skip to content

fix#3874 (http-support): make NewHttpClientImpl.execute non-blocking#3875

Open
YianZhao wants to merge 2 commits intoeclipse-sw360:mainfrom
YianZhao:bug-native-httpclient-nonblocking-issue
Open

fix#3874 (http-support): make NewHttpClientImpl.execute non-blocking#3875
YianZhao wants to merge 2 commits intoeclipse-sw360:mainfrom
YianZhao:bug-native-httpclient-nonblocking-issue

Conversation

@YianZhao
Copy link
Contributor

Implemented a fix for the native HTTP client async contract in NewHttpClientImpl.execute().

  • Removed blocking behavior (sendAsync(...).get()) and replaced it with a non-blocking async chain.
  • execute() now returns immediately and completes the returned CompletableFuture when the HTTP response arrives.
  • Preserved exception propagation from ResponseProcessor in async completion flow.
  • Added dedicated unit tests for non-blocking behavior and processor exception path.

Issue: #3874

No new dependencies were added or updated.

Suggest Reviewer

@GMishx

How To Test?

  1. Run:
    mvn -pl clients/http-support "-Dbase.deploy.dir=." "-DskipTests=false" "-Dmaven.test.skip=false" "-Dtest=NewHttpClientImplTest,HttpClientImplTest" test

@GMishx GMishx added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for labels Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs code review needs general test This is general testing, meaning that there is no org specific issue to check for

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants