slachiewicz opened a new pull request, #368: URL: https://github.com/apache/maven-jxr/pull/368
The plugin's only use of Wagon was `getStructure`, which built an `org.apache.maven.wagon.repository.Repository` to split a distributionManagement site URL into host plus path — the staging directory `site:stage` writes into. `java.net.URI` does that, once the SCM prefix is peeled off. Two behaviour notes: - A `dav:` URL now keeps its real host. Wagon parsed `dav:https://maven.apache.org/plugins/x` into host `https` and base directory `.apache.org/plugins/x`, so the staged path was nonsense. The `dav:` prefix is now peeled off the way `scm:svn:` always was. - A site URL that is not a URI at all is now reported rather than silently yielding a garbage path. There were no tests on this method. The new ones pin the values Wagon produced for every URL form it handled — `https` with and without a trailing slash, no path, an explicit port, `scm:svn:`, `scp://user@host`, `file:` — so the replacement is checked against the old parser rather than against my reading of it. *This change was created with AI assistance.* -- 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]
