Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.4
    • Labels:
      None
    • Environment:
      maven rc2
    • Number of attachments :
      0

      Description

      The linkchecker flows MOVED_TEMPORARILY moved links but sikps MOVED_PERMANENTLY while there are as valid as MOVED_TEMPORARILY

      example:
      http://www.mmbase.org/license

      resolution:
      add
      import org.apache.commons.httpclient.HttpStatus;
      to the class imports

      in HTTPLinkValidator at row 108 change the code to
      if (gm.getStatusCode() == HttpStatus.SC_MOVED_TEMPORARILY || gm.getStatusCode() == HttpStatus.SC_MOVED_PERMANENTLY)

      (and a small cleanup )
      row 121
      if (gm.getStatusCode() == HttpStatus.SC_OK)

        Activity

        Hide
        Carlos Sanchez added a comment -

        IMHO is better to report MOVED PERMANENTLY as errors so the user can change the links.

        Show
        Carlos Sanchez added a comment - IMHO is better to report MOVED PERMANENTLY as errors so the user can change the links.
        Hide
        Kees Jongenburger added a comment -

        I partialy agree. following moved_permanently should not be automatic
        Still it's a valid url

        Show
        Kees Jongenburger added a comment - I partialy agree. following moved_permanently should not be automatic Still it's a valid url
        Hide
        Arnaud Heritier added a comment -

        Fixed with MPLINKCHECK-25

        Show
        Arnaud Heritier added a comment - Fixed with MPLINKCHECK-25

          People

          • Assignee:
            Arnaud Heritier
            Reporter:
            Kees Jongenburger
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 10 minutes
              10m
              Remaining:
              Remaining Estimate - 10 minutes
              10m
              Logged:
              Time Spent - Not Specified
              Not Specified