robertlazarski commented on PR #1318:
URL:
https://github.com/apache/axis-axis2-java-core/pull/1318#issuecomment-6404724337
Thanks, this is a good catch and the query-string fix looks right. Two
spots still follow the same pattern, so it would be great to close them in this
PR: getOperationName still uses
lastIndexOf(serviceName + "/"), so POST
/axis2/services/Echo/echo/Echo/deleteAll still reaches deleteAll (searching for
the operation only after the service-path index should fix it), and
HTTPLocationBasedDispatcher.parseRequestURL has its own lastIndexOf over
the full To address including the query, so GET
/axis2/services/Svc/safe?x=/Svc/delete resolves /delete. A test
for each, and a status-code check in RequestURIDispatchITCase, and I think
this is ready.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
