Skip to content

CXF-8616: Calling oneway methods using async client hangs response indefinitely#872

Merged
reta merged 1 commit intoapache:masterfrom
reta:CXF-8616
Nov 9, 2021
Merged

CXF-8616: Calling oneway methods using async client hangs response indefinitely#872
reta merged 1 commit intoapache:masterfrom
reta:CXF-8616

Conversation

@reta
Copy link
Member

@reta reta commented Nov 7, 2021

It turns out that when async client is used with oneway methods (in CXF context, oneway is the method that returns 202), the response hangs indefinitely due to the fact that the client callback is never called / propagated. Additionally, it turned out, for sync / async flows, the response filters are also not called for oneway methods.

  • Introduced new contextual property PROPAGATE_202_RESPONSE_ONEWAY_OR_PARTIAL
  • It is set to false by default but JAX-RS clients override it with true

@reta reta force-pushed the CXF-8616 branch 2 times, most recently from 0bc1422 to c7d2906 Compare November 7, 2021 22:49
@reta
Copy link
Member Author

reta commented Nov 9, 2021

@ffang sorry but would appreciate if you could validate this one for me as well, I run into this issue while adding test cases for HttpClient 5.x support, thank you

@reta reta requested a review from ffang November 9, 2021 16:52
@ffang
Copy link
Contributor

ffang commented Nov 9, 2021

@ffang sorry but would appreciate if you could validate this one for me as well, I run into this issue while adding test cases for HttpClient 5.x support, thank you

Hi @reta ,

Looks good to me.

Could you please add affected component in the original JIRA ticket as "JAXRS" since this only affects JAXRS service?

Thanks!
Freeman

@reta reta merged commit b2eafde into apache:master Nov 9, 2021
reta added a commit that referenced this pull request Nov 10, 2021
reta added a commit that referenced this pull request Nov 10, 2021
…definitely (#872)

(cherry picked from commit b2eafde)
(cherry picked from commit fc916e1)

# Conflicts:
#	systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java
#	systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerBookTest.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants