[maven-javadoc-plugin] branch master updated: [MJAVADOC-607] followLinks fix to get redirect target breaks for certain sites

[rfscholte]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new b1c029f  [MJAVADOC-607] followLinks fix to get redirect target breaks 
for certain sites
b1c029f is described below

commit b1c029f26c4dc40b29a0e122e5c42b294845acfd
Author: rfscholte <rfscho...@apache.org>
AuthorDate: Sun Jul 28 20:51:35 2019 +0200

    [MJAVADOC-607] followLinks fix to get redirect target breaks for certain 
sites
    
    Signed-off-by: Doychin Bondzhev <doyc...@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..9250cd0 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;