From f392e3f8ad845d71426bcae269ce591810edbcd3 Mon Sep 17 00:00:00 2001 From: aheritier Date: Thu, 30 Mar 2006 23:11:50 +0000 Subject: [PATCH] Add a warning for all moves (temporary or not) git-svn-id: https://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk@390276 13f79535-47bb-0310-9956-ffa450edef68 --- .../validation/OnlineHTTPLinkValidator.java | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/linkcheck/src/main/org/apache/maven/plugin/linkcheck/validation/OnlineHTTPLinkValidator.java b/linkcheck/src/main/org/apache/maven/plugin/linkcheck/validation/OnlineHTTPLinkValidator.java index 23455664..10a1f38d 100644 --- a/linkcheck/src/main/org/apache/maven/plugin/linkcheck/validation/OnlineHTTPLinkValidator.java +++ b/linkcheck/src/main/org/apache/maven/plugin/linkcheck/validation/OnlineHTTPLinkValidator.java @@ -138,7 +138,10 @@ public final class OnlineHTTPLinkValidator } else { - if ( hm.getStatusCode() == HttpStatus.SC_MOVED_PERMANENTLY ) + // If there's a redirection ... add a warning + if ( hm.getStatusCode() == HttpStatus.SC_MOVED_PERMANENTLY + || hm.getStatusCode() == HttpStatus.SC_MOVED_TEMPORARILY + || hm.getStatusCode() == HttpStatus.SC_TEMPORARY_REDIRECT ) { LOG.warn( "Received: [" + hm.getStatusCode() + "] for [" + lvi.getLink() + "] in page [" + lvi.getSource() + "]" ); @@ -264,11 +267,11 @@ public final class OnlineHTTPLinkValidator } HttpMethod oldHm = hm; LOG.info( "[" + link + "] is redirected to [" + newLink + "]" ); - hm.releaseConnection(); + oldHm.releaseConnection(); hm = checkLink( cl, newLink, nbRedirect + 1 ); - // Restore the hm to "Moved permanently" if the new location is found to allow us to report it - if ( oldHm.getStatusCode() == HttpStatus.SC_MOVED_PERMANENTLY - && hm.getStatusCode() == HttpStatus.SC_OK ) + // Restore the hm to "Moved permanently" | "Moved temporarily" | "Temporary redirect" + // if the new location is found to allow us to report it + if ( hm.getStatusCode() == HttpStatus.SC_OK && nbRedirect == 0) { return oldHm; }