Skip to content

Commit

Permalink
[MJAVADOC-607] followLinks fix to get redirect target breaks for cert…
Browse files Browse the repository at this point in the history
…ain sites

Signed-off-by: Doychin Bondzhev <doychin@dsoft-bg.com>
  • Loading branch information
rfscholte committed Jul 28, 2019
1 parent 1969cea commit b1c029f
Showing 1 changed file with 5 additions and 0 deletions.
Expand Up @@ -6345,6 +6345,11 @@ private Set<String> followLinks( Set<String> links )
{
// 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;
Expand Down

0 comments on commit b1c029f

Please sign in to comment.