[
https://issues.apache.org/jira/browse/MJAVADOC-527?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16700594#comment-16700594
]
ASF GitHub Bot commented on MJAVADOC-527:
-----------------------------------------
dedabob commented on issue #4: [MJAVADOC-527] - detectLinks may pass invalid
urls to javadoc
URL:
https://github.com/apache/maven-javadoc-plugin/pull/4#issuecomment-442102471
Verify we were redirected to a location ending with `/package-list`. This is
faster then using `validateLinks`, which can be used anyway to verify the
content of the resource.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> detectLinks may pass invalid urls to javadoc
> --------------------------------------------
>
> Key: MJAVADOC-527
> URL: https://issues.apache.org/jira/browse/MJAVADOC-527
> Project: Maven Javadoc Plugin
> Issue Type: Bug
> Components: javadoc
> Affects Versions: 3.0.1
> Environment: Windows 10
> JDK 8
> Maven 3.5.2
> Reporter: Roberto Benedetti
> Priority: Major
> Labels: detectLinks
>
> The url of artifact com.sun.mail:mailapi:1.5.5 is
> [http://javamail.java.net/mailapi], so the plugin tests if
> [http://javamail.java.net/mailapi/apidocs/package-list] is valid.
> That url redirects to [https://javaee.github.io/javamail/] which is JavaMail
> home page, so the plugin thinks the url is valid and passes it to javadoc.
> javadoc warns about invalid link.
> Maybe checking if the effective url is still "package-list" would be safer.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
