Re: [PR] make buildOpenApiUrl return a URI [axis-axis2-jav a-core]

rootvector2 commented on PR #1242:
URL: https://github.com/apache/axis-axis2-java-core/pull/1242#issuecomment-4901237332

   fair enough on the threat model. reworked it along the lines you suggested: `buildOpenApiUrl` now returns a `java.net.URI` that gets passed around until it's embedded in the html, so the url is parsed and validated at construction instead of being a raw string. dropped `escapeJs` and its test, and updated the title/body to match. `mvn -pl modules/openapi test` passes.


-- 
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]