You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by rf...@apache.org on 2019/07/28 13:55:09 UTC

[maven-javadoc-plugin] 01/01: Fix for MJAVADOC-607

This is an automated email from the ASF dual-hosted git repository.

rfscholte pushed a commit to branch MJAVADOC-607
in repository https://gitbox.apache.org/repos/asf/maven-javadoc-plugin.git

commit 786c5a1872d020f3113044bf9af844a8e06d9449
Author: Doychin Bondzhev <do...@dsoft-bg.com>
AuthorDate: Fri Jul 19 11:37:32 2019 +0300

    Fix for MJAVADOC-607
    
    Signed-off-by: Doychin Bondzhev <do...@dsoft-bg.com>
---
 .../java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java   | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java b/src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java
index cb68f86..b38e315 100644
--- a/src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java
+++ b/src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java
@@ -6345,6 +6345,11 @@ public abstract class AbstractJavadocMojo
             {
                 // only print in debug, it should have been logged already in warn/error because link isn't valid
                 getLog().debug( "Could not follow " + link + ". Reason: " + e.getMessage() );
+
+                // Even when link produces error it should be kept in the set because the error might be caused by
+                // incomplete redirect configuration on the server side.
+                // This partially restores the previous behaviour before fix for MJAVADOC-427
+                redirectLinks.add(link);
             }
         }
         return redirectLinks;