Archiva
  1. Archiva
  2. MRM-893

Illegal browse operation allow

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.1
    • Fix Version/s: 1.1.2
    • Component/s: browser
    • Labels:
      None
    • Patch Submitted:
      Yes
    • Number of attachments :
      3

      Description

      http://archiva.openqa.org/repository/releases/org/openqa/selenium/client-drivers/selenium-client-drivers/0.9.2/selenium-client-drivers-0.9.2-tests.jar/ <-- note the last forward slash

      return the jar which is illegal

      http://repo1.maven.org/maven2/HTTPClient/HTTPClient/0.3-3/HTTPClient-0.3-3.jar/

      return http 404 which is correct.

      This bug crashes maven-staging-plugin with wagon-beta-3 since it tries to pass the content of the jar to
      the html parser and crashes with ArrayOutOfIndex exception.

      see http://jira.codehaus.org/browse/MRM-891 for details.

      Sorry I am not able provide patch at this moment.

      1. MRM-893.patch
        2 kB
        Dan Tran
      2. MRM-893-2.patch
        3 kB
        Dan Tran
      3. MRM-893-3.patch
        3 kB
        Dan Tran

        Activity

        Hide
        Brett Porter added a comment -

        nice catch! If you're able to test and submit a patch that'd be most helpful in applying it to the next release

        Show
        Brett Porter added a comment - nice catch! If you're able to test and submit a patch that'd be most helpful in applying it to the next release
        Hide
        Dan Tran added a comment -

        attached is the fix + test. I found a better way to handle this specific condition without removing the intended behavior of purposely removing the end slashed when computing the local path

        Show
        Dan Tran added a comment - attached is the fix + test. I found a better way to handle this specific condition without removing the intended behavior of purposely removing the end slashed when computing the local path
        Hide
        Dan Tran added a comment -

        MRM-893-2.patch has a better test coverage

        Show
        Dan Tran added a comment - MRM-893 -2.patch has a better test coverage
        Hide
        Dan Tran added a comment -

        more tests, just to make sure i dont break any thing. this is it.

        Show
        Dan Tran added a comment - more tests, just to make sure i dont break any thing. this is it.
        Hide
        Maria Odea Ching added a comment -

        Patch applied in trunk -r685076. Thanks Dan!

        Show
        Maria Odea Ching added a comment - Patch applied in trunk -r685076. Thanks Dan!

          People

          • Assignee:
            Maria Odea Ching
            Reporter:
            Dan Tran
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: