Skip to content

stop storing the wsdl request address on the shared AxisService - #1313

Merged
robertlazarski merged 1 commit into
apache:masterfrom
kali834x:wsdl-request-address
Oct 20, 2026
Merged

robertlazarski merged 1 commit into
apache:masterfrom
kali834x:wsdl-request-address

Conversation

@kali834x

Copy link
Copy Markdown
Contributor

Request host written into the shared AxisService during WSDL generation
WSDLDataLocator.outputInlineForm hands the requester's address to AxisService2WSDL11 by storing it in AxisService.endpointURL, a field every request on the deployed service shares, and MexMessageReceiver stores the inbound wsa:To there verbatim. getEndpointURL reads the host back out of that field for each soap:address, so two concurrent ?wsdl fetches race on it and one client's Host header lands in the WSDL another client receives (9 of 800 in a two-thread reproducer), and a GetMetadata request leaves its To on the service for later generations. The address now travels with the generator for that one request; the hostname parameter and modifyUserWSDLPortAddress behave as before. The kernel test fails before the change.

@robertlazarski

Copy link
Copy Markdown
Contributor

@kali834x this is another example of good work and I have a question for you: are you interested in becoming an Apache Axis2 committer?

Feel free to continue sending PR's if not.

As a committer - automatically a PMC role with voting rights in this project - you can merge them yourself.

I ask because there is a vote in progress right now for Axis2/Java 2.0.2, and Apache projects have a strict three person rule as active.

However, one of our reliable committers - who merged one of your previous PRs - is not present at the moment.

It highlights that Axis2 needs more committers. There's not much responsibility besides occasionally voting.

Let me know if interested.

I am merging your open PR's in the meantime, following a review.

@robertlazarski
robertlazarski merged commit 47187d4 into apache:master Oct 20, 2026
@kali834x

Copy link
Copy Markdown
Contributor Author

Thank you, I’d be happy to join.

@alphatheory-rlazarski

Copy link
Copy Markdown
Contributor

Thanks, that's great to hear. Next step is a vote among the Axis PMC, which happens on our private mailing list and normally runs about a week. Before I start it, please email me your full legal name at robertlazarski@apache.org (it goes on the ICLA, the contributor agreement every committer signs). If the vote passes I'll send a formal invitation with the ICLA instructions and the account request steps.

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.

3 participants