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

Client using SOAP 1.2 sends no SOAP Action #1640

Closed
CodedNikls opened this issue Dec 4, 2024 · 3 comments
Closed

Client using SOAP 1.2 sends no SOAP Action #1640

CodedNikls opened this issue Dec 4, 2024 · 3 comments

Comments

@CodedNikls
Copy link

CodedNikls commented Dec 4, 2024

I configured my quarkus cxf client to use SOAP 1.2 via the soap-binding property like this:

quarkus.cxf.client.myClient.soap-binding=http://www.w3.org/2003/05/soap/bindings/HTTP/

The client is set up using the @CXFClient annotation and an interface generated from a wsdl file, which has been working so far.
Before, the messages it sent out used to have a Header "SOAPAction", now they don't.

Is there a solution to this I am missing or is this a bug?
Thank you in advance!

@ppalaga
Copy link
Contributor

ppalaga commented Dec 4, 2024

Thanks for the report @CodedNikls!

I was not able to get the SOAPAction in the request with neither the old URLConnectionHTTPConduitFactory neither with the new default VertxHttpClientHTTPConduitFactory

Could you perhaps try to change my test in such a way that it shows the "old" behavior (once you uncomment the quarkus.cxf.client.soap12.http-conduit-factory = URLConnectionHTTPConduitFactory line)?

https://github.com/quarkiverse/quarkus-cxf/pull/1644/files#diff-0f3918539c30097c2c99b110b24d1d460c20e3263a07244a54f5463a4e7b3273R126-R130

@CodedNikls
Copy link
Author

Thank you for the answer @ppalaga! After further investigation, we have discovered that a SOAPAction header is not required in SOAP 1.2 anymore and the soap action has instead been moved to the content-type in format
"Content-Type: application/soap+xml; charset=utf-8; action="[...]""
So, right now we are stuck on the issue regarding the content-type and will need to investigate the Soap action there as well.

@ppalaga
Copy link
Contributor

ppalaga commented Dec 11, 2024

Thanks for the feedback, @CodedNikls. Let's focus on #1641 then.

@ppalaga ppalaga closed this as not planned Won't fix, can't repro, duplicate, stale Dec 11, 2024
@ppalaga ppalaga added this to the no fix/won't fix milestone Dec 11, 2024
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

No branches or pull requests

2 participants