You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Christian Schulte (JIRA)" <ji...@codehaus.org> on 2010/05/02 08:18:15 UTC
[jira] Created: (MJAVADOC-283) isValidJavadocLink should be more
strict
isValidJavadocLink should be more strict
----------------------------------------
Key: MJAVADOC-283
URL: http://jira.codehaus.org/browse/MJAVADOC-283
Project: Maven 2.x Javadoc Plugin
Issue Type: Improvement
Affects Versions: 2.6.1
Environment: Apache Maven 2.2.1 (r801777; 2009-08-06 21:16:01+0200)
Java version: 1.5.0_16-p9
Java home: /usr/local/jdk-1.5.0/jre
Default locale: de_DE, platform encoding: ISO8859-15
OS name: "openbsd" version: "4.6" arch: "i386" Family: "unix"
Reporter: Christian Schulte
Priority: Blocker
When setting 'detectLinks' to 'true', the plugin checks that access to the constructed package-list files is possible by checking the HTTP status code to match 200. This check should be more strict and additionally check that the accessed package-list file is of correct type. For example, if setting detect links to 'true' and having a dependency on 'javax.annotation:jsr250-api' the plugin will add a link like 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs' so that javadoc will then try to access the package-list file located at 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs/package-list'. That link returns a HTTP status code 200 but no valid package-list file. Using the following javadoc plugin configuration
{xml}
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-javadoc-plugin</artifactId>
<version>2.7</version>
<configuration>
<breakiterator>true</breakiterator>
<keywords>true</keywords>
<linksource>true</linksource>
<serialwarn>true</serialwarn>
<source>1.5</source>
<doctitle>${project.name} ${project.version}</doctitle>
<windowtitle>${project.name} ${project.version}</windowtitle>
<detectJavaApiLink>true</detectJavaApiLink>
<detectOfflineLinks>false</detectOfflineLinks>
<detectLinks>true</detectLinks>
<doclet>org.umlgraph.doclet.UmlGraphDoc</doclet>
<docletArtifact>
<groupId>org.umlgraph</groupId>
<artifactId>doclet</artifactId>
<version>5.1</version>
</docletArtifact>
<tagletArtifacts>
<tagletArtifact>
<groupId>org.apache.maven.plugin-tools</groupId>
<artifactId>maven-plugin-tools-javadoc</artifactId>
<version>2.5</version>
</tagletArtifact>
<tagletArtifact>
<groupId>org.codehaus.plexus</groupId>
<artifactId>plexus-javadoc</artifactId>
<version>1.0</version>
</tagletArtifact>
</tagletArtifacts>
<additionalparam>
-inferrel -inferdep -hide java.* -collpackages java.util.* -qualify -postfixpackage -nodefontsize 9
-nodefontpackagesize 7
</additionalparam>
<archive>
<index>true</index>
<addMavenDescriptor>false</addMavenDescriptor>
<compress>true</compress>
</archive>
</configuration>
</plugin>
{xml}
the configured umlgraph doclet will fail when trying to parse the retrieved HTML document as a package-list file.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Updated: (MJAVADOC-283) isValidJavadocLink should be more
strict
Posted by "Christian Schulte (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MJAVADOC-283?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Christian Schulte updated MJAVADOC-283:
---------------------------------------
Attachment: MJAVADOC-283.patch
Patch adding a 'validateLinks' option which will cause the plugin to download and validate the content of 'package-list' resources.
> isValidJavadocLink should be more strict
> ----------------------------------------
>
> Key: MJAVADOC-283
> URL: http://jira.codehaus.org/browse/MJAVADOC-283
> Project: Maven 2.x Javadoc Plugin
> Issue Type: Improvement
> Affects Versions: 2.6.1
> Environment: Apache Maven 2.2.1 (r801777; 2009-08-06 21:16:01+0200)
> Java version: 1.5.0_16-p9
> Java home: /usr/local/jdk-1.5.0/jre
> Default locale: de_DE, platform encoding: ISO8859-15
> OS name: "openbsd" version: "4.6" arch: "i386" Family: "unix"
> Reporter: Christian Schulte
> Priority: Blocker
> Attachments: MJAVADOC-283.patch, MJAVADOC-283.patch
>
>
> When setting 'detectLinks' to 'true', the plugin checks that access to the constructed package-list files is possible by checking the HTTP status code to match 200. This check should be more strict and additionally check that the accessed package-list file is of correct type. For example, if setting detect links to 'true' and having a dependency on 'javax.annotation:jsr250-api' the plugin will add a link like 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs' so that javadoc will then try to access the package-list file located at 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs/package-list'. That link returns a HTTP status code 200 but no valid package-list file. Using the following javadoc plugin configuration
> {code:xml}
> <plugin>
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-javadoc-plugin</artifactId>
> <version>2.7</version>
> <configuration>
> <breakiterator>true</breakiterator>
> <keywords>true</keywords>
> <linksource>true</linksource>
> <serialwarn>true</serialwarn>
> <source>1.5</source>
> <doctitle>${project.name} ${project.version}</doctitle>
> <windowtitle>${project.name} ${project.version}</windowtitle>
> <detectJavaApiLink>true</detectJavaApiLink>
> <detectOfflineLinks>false</detectOfflineLinks>
> <detectLinks>true</detectLinks>
> <doclet>org.umlgraph.doclet.UmlGraphDoc</doclet>
> <docletArtifact>
> <groupId>org.umlgraph</groupId>
> <artifactId>doclet</artifactId>
> <version>5.1</version>
> </docletArtifact>
> <tagletArtifacts>
> <tagletArtifact>
> <groupId>org.apache.maven.plugin-tools</groupId>
> <artifactId>maven-plugin-tools-javadoc</artifactId>
> <version>2.5</version>
> </tagletArtifact>
> <tagletArtifact>
> <groupId>org.codehaus.plexus</groupId>
> <artifactId>plexus-javadoc</artifactId>
> <version>1.0</version>
> </tagletArtifact>
> </tagletArtifacts>
> <additionalparam>
> -inferrel -inferdep -hide java.* -collpackages java.util.* -qualify -postfixpackage -nodefontsize 9
> -nodefontpackagesize 7
> </additionalparam>
> <archive>
> <index>true</index>
> <addMavenDescriptor>false</addMavenDescriptor>
> <compress>true</compress>
> </archive>
> </configuration>
> </plugin>
> {code}
>
> the configured umlgraph doclet will fail when trying to parse the retrieved HTML document as a package-list file.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Updated: (MJAVADOC-283) isValidJavadocLink should be more
strict
Posted by "Herve Boutemy (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MJAVADOC-283?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Herve Boutemy updated MJAVADOC-283:
-----------------------------------
Description:
When setting 'detectLinks' to 'true', the plugin checks that access to the constructed package-list files is possible by checking the HTTP status code to match 200. This check should be more strict and additionally check that the accessed package-list file is of correct type. For example, if setting detect links to 'true' and having a dependency on 'javax.annotation:jsr250-api' the plugin will add a link like 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs' so that javadoc will then try to access the package-list file located at 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs/package-list'. That link returns a HTTP status code 200 but no valid package-list file. Using the following javadoc plugin configuration
{code:xml}
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-javadoc-plugin</artifactId>
<version>2.7</version>
<configuration>
<breakiterator>true</breakiterator>
<keywords>true</keywords>
<linksource>true</linksource>
<serialwarn>true</serialwarn>
<source>1.5</source>
<doctitle>${project.name} ${project.version}</doctitle>
<windowtitle>${project.name} ${project.version}</windowtitle>
<detectJavaApiLink>true</detectJavaApiLink>
<detectOfflineLinks>false</detectOfflineLinks>
<detectLinks>true</detectLinks>
<doclet>org.umlgraph.doclet.UmlGraphDoc</doclet>
<docletArtifact>
<groupId>org.umlgraph</groupId>
<artifactId>doclet</artifactId>
<version>5.1</version>
</docletArtifact>
<tagletArtifacts>
<tagletArtifact>
<groupId>org.apache.maven.plugin-tools</groupId>
<artifactId>maven-plugin-tools-javadoc</artifactId>
<version>2.5</version>
</tagletArtifact>
<tagletArtifact>
<groupId>org.codehaus.plexus</groupId>
<artifactId>plexus-javadoc</artifactId>
<version>1.0</version>
</tagletArtifact>
</tagletArtifacts>
<additionalparam>
-inferrel -inferdep -hide java.* -collpackages java.util.* -qualify -postfixpackage -nodefontsize 9
-nodefontpackagesize 7
</additionalparam>
<archive>
<index>true</index>
<addMavenDescriptor>false</addMavenDescriptor>
<compress>true</compress>
</archive>
</configuration>
</plugin>
{code}
the configured umlgraph doclet will fail when trying to parse the retrieved HTML document as a package-list file.
was:
When setting 'detectLinks' to 'true', the plugin checks that access to the constructed package-list files is possible by checking the HTTP status code to match 200. This check should be more strict and additionally check that the accessed package-list file is of correct type. For example, if setting detect links to 'true' and having a dependency on 'javax.annotation:jsr250-api' the plugin will add a link like 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs' so that javadoc will then try to access the package-list file located at 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs/package-list'. That link returns a HTTP status code 200 but no valid package-list file. Using the following javadoc plugin configuration
{xml}
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-javadoc-plugin</artifactId>
<version>2.7</version>
<configuration>
<breakiterator>true</breakiterator>
<keywords>true</keywords>
<linksource>true</linksource>
<serialwarn>true</serialwarn>
<source>1.5</source>
<doctitle>${project.name} ${project.version}</doctitle>
<windowtitle>${project.name} ${project.version}</windowtitle>
<detectJavaApiLink>true</detectJavaApiLink>
<detectOfflineLinks>false</detectOfflineLinks>
<detectLinks>true</detectLinks>
<doclet>org.umlgraph.doclet.UmlGraphDoc</doclet>
<docletArtifact>
<groupId>org.umlgraph</groupId>
<artifactId>doclet</artifactId>
<version>5.1</version>
</docletArtifact>
<tagletArtifacts>
<tagletArtifact>
<groupId>org.apache.maven.plugin-tools</groupId>
<artifactId>maven-plugin-tools-javadoc</artifactId>
<version>2.5</version>
</tagletArtifact>
<tagletArtifact>
<groupId>org.codehaus.plexus</groupId>
<artifactId>plexus-javadoc</artifactId>
<version>1.0</version>
</tagletArtifact>
</tagletArtifacts>
<additionalparam>
-inferrel -inferdep -hide java.* -collpackages java.util.* -qualify -postfixpackage -nodefontsize 9
-nodefontpackagesize 7
</additionalparam>
<archive>
<index>true</index>
<addMavenDescriptor>false</addMavenDescriptor>
<compress>true</compress>
</archive>
</configuration>
</plugin>
{xml}
the configured umlgraph doclet will fail when trying to parse the retrieved HTML document as a package-list file.
> isValidJavadocLink should be more strict
> ----------------------------------------
>
> Key: MJAVADOC-283
> URL: http://jira.codehaus.org/browse/MJAVADOC-283
> Project: Maven 2.x Javadoc Plugin
> Issue Type: Improvement
> Affects Versions: 2.6.1
> Environment: Apache Maven 2.2.1 (r801777; 2009-08-06 21:16:01+0200)
> Java version: 1.5.0_16-p9
> Java home: /usr/local/jdk-1.5.0/jre
> Default locale: de_DE, platform encoding: ISO8859-15
> OS name: "openbsd" version: "4.6" arch: "i386" Family: "unix"
> Reporter: Christian Schulte
> Priority: Blocker
> Attachments: MJAVADOC-283.patch
>
>
> When setting 'detectLinks' to 'true', the plugin checks that access to the constructed package-list files is possible by checking the HTTP status code to match 200. This check should be more strict and additionally check that the accessed package-list file is of correct type. For example, if setting detect links to 'true' and having a dependency on 'javax.annotation:jsr250-api' the plugin will add a link like 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs' so that javadoc will then try to access the package-list file located at 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs/package-list'. That link returns a HTTP status code 200 but no valid package-list file. Using the following javadoc plugin configuration
> {code:xml}
> <plugin>
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-javadoc-plugin</artifactId>
> <version>2.7</version>
> <configuration>
> <breakiterator>true</breakiterator>
> <keywords>true</keywords>
> <linksource>true</linksource>
> <serialwarn>true</serialwarn>
> <source>1.5</source>
> <doctitle>${project.name} ${project.version}</doctitle>
> <windowtitle>${project.name} ${project.version}</windowtitle>
> <detectJavaApiLink>true</detectJavaApiLink>
> <detectOfflineLinks>false</detectOfflineLinks>
> <detectLinks>true</detectLinks>
> <doclet>org.umlgraph.doclet.UmlGraphDoc</doclet>
> <docletArtifact>
> <groupId>org.umlgraph</groupId>
> <artifactId>doclet</artifactId>
> <version>5.1</version>
> </docletArtifact>
> <tagletArtifacts>
> <tagletArtifact>
> <groupId>org.apache.maven.plugin-tools</groupId>
> <artifactId>maven-plugin-tools-javadoc</artifactId>
> <version>2.5</version>
> </tagletArtifact>
> <tagletArtifact>
> <groupId>org.codehaus.plexus</groupId>
> <artifactId>plexus-javadoc</artifactId>
> <version>1.0</version>
> </tagletArtifact>
> </tagletArtifacts>
> <additionalparam>
> -inferrel -inferdep -hide java.* -collpackages java.util.* -qualify -postfixpackage -nodefontsize 9
> -nodefontpackagesize 7
> </additionalparam>
> <archive>
> <index>true</index>
> <addMavenDescriptor>false</addMavenDescriptor>
> <compress>true</compress>
> </archive>
> </configuration>
> </plugin>
> {code}
>
> the configured umlgraph doclet will fail when trying to parse the retrieved HTML document as a package-list file.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Closed: (MJAVADOC-283) isValidJavadocLink should be more
strict
Posted by "Herve Boutemy (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MJAVADOC-283?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Herve Boutemy closed MJAVADOC-283.
----------------------------------
Resolution: Fixed
Fix Version/s: 2.8
Assignee: Herve Boutemy
patch applied in [r1096493|http://svn.apache.org/viewvc?rev=1096493&view=rev] with updated unit-tests
thank you
> isValidJavadocLink should be more strict
> ----------------------------------------
>
> Key: MJAVADOC-283
> URL: http://jira.codehaus.org/browse/MJAVADOC-283
> Project: Maven 2.x Javadoc Plugin
> Issue Type: Improvement
> Affects Versions: 2.6.1
> Environment: Apache Maven 2.2.1 (r801777; 2009-08-06 21:16:01+0200)
> Java version: 1.5.0_16-p9
> Java home: /usr/local/jdk-1.5.0/jre
> Default locale: de_DE, platform encoding: ISO8859-15
> OS name: "openbsd" version: "4.6" arch: "i386" Family: "unix"
> Reporter: Christian Schulte
> Assignee: Herve Boutemy
> Priority: Blocker
> Fix For: 2.8
>
> Attachments: MJAVADOC-283.patch, MJAVADOC-283.patch
>
>
> When setting 'detectLinks' to 'true', the plugin checks that access to the constructed package-list files is possible by checking the HTTP status code to match 200. This check should be more strict and additionally check that the accessed package-list file is of correct type. For example, if setting detect links to 'true' and having a dependency on 'javax.annotation:jsr250-api' the plugin will add a link like 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs' so that javadoc will then try to access the package-list file located at 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs/package-list'. That link returns a HTTP status code 200 but no valid package-list file. Using the following javadoc plugin configuration
> {code:xml}
> <plugin>
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-javadoc-plugin</artifactId>
> <version>2.7</version>
> <configuration>
> <breakiterator>true</breakiterator>
> <keywords>true</keywords>
> <linksource>true</linksource>
> <serialwarn>true</serialwarn>
> <source>1.5</source>
> <doctitle>${project.name} ${project.version}</doctitle>
> <windowtitle>${project.name} ${project.version}</windowtitle>
> <detectJavaApiLink>true</detectJavaApiLink>
> <detectOfflineLinks>false</detectOfflineLinks>
> <detectLinks>true</detectLinks>
> <doclet>org.umlgraph.doclet.UmlGraphDoc</doclet>
> <docletArtifact>
> <groupId>org.umlgraph</groupId>
> <artifactId>doclet</artifactId>
> <version>5.1</version>
> </docletArtifact>
> <tagletArtifacts>
> <tagletArtifact>
> <groupId>org.apache.maven.plugin-tools</groupId>
> <artifactId>maven-plugin-tools-javadoc</artifactId>
> <version>2.5</version>
> </tagletArtifact>
> <tagletArtifact>
> <groupId>org.codehaus.plexus</groupId>
> <artifactId>plexus-javadoc</artifactId>
> <version>1.0</version>
> </tagletArtifact>
> </tagletArtifacts>
> <additionalparam>
> -inferrel -inferdep -hide java.* -collpackages java.util.* -qualify -postfixpackage -nodefontsize 9
> -nodefontpackagesize 7
> </additionalparam>
> <archive>
> <index>true</index>
> <addMavenDescriptor>false</addMavenDescriptor>
> <compress>true</compress>
> </archive>
> </configuration>
> </plugin>
> {code}
>
> the configured umlgraph doclet will fail when trying to parse the retrieved HTML document as a package-list file.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Updated: (MJAVADOC-283) isValidJavadocLink should be more
strict
Posted by "Christian Schulte (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MJAVADOC-283?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Christian Schulte updated MJAVADOC-283:
---------------------------------------
Attachment: MJAVADOC-283.patch
Patch adding a check of the HTTP Content-Type response header when validating javadoc links.
> isValidJavadocLink should be more strict
> ----------------------------------------
>
> Key: MJAVADOC-283
> URL: http://jira.codehaus.org/browse/MJAVADOC-283
> Project: Maven 2.x Javadoc Plugin
> Issue Type: Improvement
> Affects Versions: 2.6.1
> Environment: Apache Maven 2.2.1 (r801777; 2009-08-06 21:16:01+0200)
> Java version: 1.5.0_16-p9
> Java home: /usr/local/jdk-1.5.0/jre
> Default locale: de_DE, platform encoding: ISO8859-15
> OS name: "openbsd" version: "4.6" arch: "i386" Family: "unix"
> Reporter: Christian Schulte
> Priority: Blocker
> Attachments: MJAVADOC-283.patch
>
>
> When setting 'detectLinks' to 'true', the plugin checks that access to the constructed package-list files is possible by checking the HTTP status code to match 200. This check should be more strict and additionally check that the accessed package-list file is of correct type. For example, if setting detect links to 'true' and having a dependency on 'javax.annotation:jsr250-api' the plugin will add a link like 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs' so that javadoc will then try to access the package-list file located at 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs/package-list'. That link returns a HTTP status code 200 but no valid package-list file. Using the following javadoc plugin configuration
> {xml}
> <plugin>
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-javadoc-plugin</artifactId>
> <version>2.7</version>
> <configuration>
> <breakiterator>true</breakiterator>
> <keywords>true</keywords>
> <linksource>true</linksource>
> <serialwarn>true</serialwarn>
> <source>1.5</source>
> <doctitle>${project.name} ${project.version}</doctitle>
> <windowtitle>${project.name} ${project.version}</windowtitle>
> <detectJavaApiLink>true</detectJavaApiLink>
> <detectOfflineLinks>false</detectOfflineLinks>
> <detectLinks>true</detectLinks>
> <doclet>org.umlgraph.doclet.UmlGraphDoc</doclet>
> <docletArtifact>
> <groupId>org.umlgraph</groupId>
> <artifactId>doclet</artifactId>
> <version>5.1</version>
> </docletArtifact>
> <tagletArtifacts>
> <tagletArtifact>
> <groupId>org.apache.maven.plugin-tools</groupId>
> <artifactId>maven-plugin-tools-javadoc</artifactId>
> <version>2.5</version>
> </tagletArtifact>
> <tagletArtifact>
> <groupId>org.codehaus.plexus</groupId>
> <artifactId>plexus-javadoc</artifactId>
> <version>1.0</version>
> </tagletArtifact>
> </tagletArtifacts>
> <additionalparam>
> -inferrel -inferdep -hide java.* -collpackages java.util.* -qualify -postfixpackage -nodefontsize 9
> -nodefontpackagesize 7
> </additionalparam>
> <archive>
> <index>true</index>
> <addMavenDescriptor>false</addMavenDescriptor>
> <compress>true</compress>
> </archive>
> </configuration>
> </plugin>
> {xml}
>
> the configured umlgraph doclet will fail when trying to parse the retrieved HTML document as a package-list file.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (MJAVADOC-283) isValidJavadocLink should be more
strict
Posted by "Christian Schulte (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MJAVADOC-283?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=219669#action_219669 ]
Christian Schulte commented on MJAVADOC-283:
--------------------------------------------
Attached patch would break 'http://junit.org/apidocs/package-list' with content type 'text/html; charset=UTF-8' although a valid package-list file. Maybe better add parameters for excluding/including the dependencies to use with 'detectLinks'.
> isValidJavadocLink should be more strict
> ----------------------------------------
>
> Key: MJAVADOC-283
> URL: http://jira.codehaus.org/browse/MJAVADOC-283
> Project: Maven 2.x Javadoc Plugin
> Issue Type: Improvement
> Affects Versions: 2.6.1
> Environment: Apache Maven 2.2.1 (r801777; 2009-08-06 21:16:01+0200)
> Java version: 1.5.0_16-p9
> Java home: /usr/local/jdk-1.5.0/jre
> Default locale: de_DE, platform encoding: ISO8859-15
> OS name: "openbsd" version: "4.6" arch: "i386" Family: "unix"
> Reporter: Christian Schulte
> Priority: Blocker
> Attachments: MJAVADOC-283.patch
>
>
> When setting 'detectLinks' to 'true', the plugin checks that access to the constructed package-list files is possible by checking the HTTP status code to match 200. This check should be more strict and additionally check that the accessed package-list file is of correct type. For example, if setting detect links to 'true' and having a dependency on 'javax.annotation:jsr250-api' the plugin will add a link like 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs' so that javadoc will then try to access the package-list file located at 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs/package-list'. That link returns a HTTP status code 200 but no valid package-list file. Using the following javadoc plugin configuration
> {xml}
> <plugin>
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-javadoc-plugin</artifactId>
> <version>2.7</version>
> <configuration>
> <breakiterator>true</breakiterator>
> <keywords>true</keywords>
> <linksource>true</linksource>
> <serialwarn>true</serialwarn>
> <source>1.5</source>
> <doctitle>${project.name} ${project.version}</doctitle>
> <windowtitle>${project.name} ${project.version}</windowtitle>
> <detectJavaApiLink>true</detectJavaApiLink>
> <detectOfflineLinks>false</detectOfflineLinks>
> <detectLinks>true</detectLinks>
> <doclet>org.umlgraph.doclet.UmlGraphDoc</doclet>
> <docletArtifact>
> <groupId>org.umlgraph</groupId>
> <artifactId>doclet</artifactId>
> <version>5.1</version>
> </docletArtifact>
> <tagletArtifacts>
> <tagletArtifact>
> <groupId>org.apache.maven.plugin-tools</groupId>
> <artifactId>maven-plugin-tools-javadoc</artifactId>
> <version>2.5</version>
> </tagletArtifact>
> <tagletArtifact>
> <groupId>org.codehaus.plexus</groupId>
> <artifactId>plexus-javadoc</artifactId>
> <version>1.0</version>
> </tagletArtifact>
> </tagletArtifacts>
> <additionalparam>
> -inferrel -inferdep -hide java.* -collpackages java.util.* -qualify -postfixpackage -nodefontsize 9
> -nodefontpackagesize 7
> </additionalparam>
> <archive>
> <index>true</index>
> <addMavenDescriptor>false</addMavenDescriptor>
> <compress>true</compress>
> </archive>
> </configuration>
> </plugin>
> {xml}
>
> the configured umlgraph doclet will fail when trying to parse the retrieved HTML document as a package-list file.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (MJAVADOC-283) isValidJavadocLink should be more
strict
Posted by "Herve Boutemy (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MJAVADOC-283?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=250366#action_250366 ]
Herve Boutemy commented on MJAVADOC-283:
----------------------------------------
or read tha actual content downloaded and check that there a no spaces nor <: I think this heuristic should be sufficient
WDYT?
> isValidJavadocLink should be more strict
> ----------------------------------------
>
> Key: MJAVADOC-283
> URL: http://jira.codehaus.org/browse/MJAVADOC-283
> Project: Maven 2.x Javadoc Plugin
> Issue Type: Improvement
> Affects Versions: 2.6.1
> Environment: Apache Maven 2.2.1 (r801777; 2009-08-06 21:16:01+0200)
> Java version: 1.5.0_16-p9
> Java home: /usr/local/jdk-1.5.0/jre
> Default locale: de_DE, platform encoding: ISO8859-15
> OS name: "openbsd" version: "4.6" arch: "i386" Family: "unix"
> Reporter: Christian Schulte
> Priority: Blocker
> Attachments: MJAVADOC-283.patch
>
>
> When setting 'detectLinks' to 'true', the plugin checks that access to the constructed package-list files is possible by checking the HTTP status code to match 200. This check should be more strict and additionally check that the accessed package-list file is of correct type. For example, if setting detect links to 'true' and having a dependency on 'javax.annotation:jsr250-api' the plugin will add a link like 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs' so that javadoc will then try to access the package-list file located at 'http://jcp.org/aboutJava/communityprocess/final/jsr250/index.html/apidocs/package-list'. That link returns a HTTP status code 200 but no valid package-list file. Using the following javadoc plugin configuration
> {code:xml}
> <plugin>
> <groupId>org.apache.maven.plugins</groupId>
> <artifactId>maven-javadoc-plugin</artifactId>
> <version>2.7</version>
> <configuration>
> <breakiterator>true</breakiterator>
> <keywords>true</keywords>
> <linksource>true</linksource>
> <serialwarn>true</serialwarn>
> <source>1.5</source>
> <doctitle>${project.name} ${project.version}</doctitle>
> <windowtitle>${project.name} ${project.version}</windowtitle>
> <detectJavaApiLink>true</detectJavaApiLink>
> <detectOfflineLinks>false</detectOfflineLinks>
> <detectLinks>true</detectLinks>
> <doclet>org.umlgraph.doclet.UmlGraphDoc</doclet>
> <docletArtifact>
> <groupId>org.umlgraph</groupId>
> <artifactId>doclet</artifactId>
> <version>5.1</version>
> </docletArtifact>
> <tagletArtifacts>
> <tagletArtifact>
> <groupId>org.apache.maven.plugin-tools</groupId>
> <artifactId>maven-plugin-tools-javadoc</artifactId>
> <version>2.5</version>
> </tagletArtifact>
> <tagletArtifact>
> <groupId>org.codehaus.plexus</groupId>
> <artifactId>plexus-javadoc</artifactId>
> <version>1.0</version>
> </tagletArtifact>
> </tagletArtifacts>
> <additionalparam>
> -inferrel -inferdep -hide java.* -collpackages java.util.* -qualify -postfixpackage -nodefontsize 9
> -nodefontpackagesize 7
> </additionalparam>
> <archive>
> <index>true</index>
> <addMavenDescriptor>false</addMavenDescriptor>
> <compress>true</compress>
> </archive>
> </configuration>
> </plugin>
> {code}
>
> the configured umlgraph doclet will fail when trying to parse the retrieved HTML document as a package-list file.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira