Pluto
  1. Pluto
  2. PLUTO-361

Error when parsing the portal url

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.2, 1.1.4
    • Fix Version/s: 1.1.5
    • Component/s: portal driver
    • Labels:
      None
    • Environment:
      Fedora 5, Tomcat 5.0.28

      Description

      When I use the below link:

      http://localhost:8080/pluto/portal/__

      I got this error:

      java.lang.StringIndexOutOfBoundsException: String index out of range: -2
      java.lang.String.substring(String.java:1768)
      java.lang.String.substring(String.java:1735)
      org.apache.pluto.driver.url.impl.PortalURLParserImpl.decodeParameter(PortalURLParserImpl.java:357)
      org.apache.pluto.driver.url.impl.PortalURLParserImpl.parse(PortalURLParserImpl.java:158)
      org.apache.pluto.driver.core.PortalRequestContext.getRequestedPortalURL(PortalRequestContext.java:113)
      org.apache.pluto.driver.PortalDriverServlet.doGet(PortalDriverServlet.java:95)
      javax.servlet.http.HttpServlet.service(HttpServlet.java:689)
      javax.servlet.http.HttpServlet.service(HttpServlet.java:802)

      1. patch-PLUTO-361.txt
        2 kB
        Marc Veary
      2. unittest-PLUTO-361.txt
        18 kB
        Marc Veary

        Issue Links

          Activity

          Hide
          Marc Veary added a comment -

          This patch fixes the issue where an action request (i.e. prefixed with '__') is made but there are no parameters to decode.

          Show
          Marc Veary added a comment - This patch fixes the issue where an action request (i.e. prefixed with '__') is made but there are no parameters to decode.
          Hide
          Marc Veary added a comment -

          The attached Unit Test was used to reproduce the issue and to test fix

          Show
          Marc Veary added a comment - The attached Unit Test was used to reproduce the issue and to test fix
          Hide
          Craig Doremus added a comment -

          Fixed applied in SVN revision 554206 in trunk and revision 554210 in pluto-1.1.x branch. Thank you Marc for the patch!

          Show
          Craig Doremus added a comment - Fixed applied in SVN revision 554206 in trunk and revision 554210 in pluto-1.1.x branch. Thank you Marc for the patch!
          Hide
          Elliot Metsger added a comment -

          This patch at least partially reverts the patch from PLUTO-356.

          Show
          Elliot Metsger added a comment - This patch at least partially reverts the patch from PLUTO-356 .
          Hide
          Elliot Metsger added a comment -

          Updating fix ver to 1.1.5.

          Show
          Elliot Metsger added a comment - Updating fix ver to 1.1.5.
          Hide
          Elliot Metsger added a comment -

          Fix committed to 1.1.x branch 609238.

          The 286 merge branch is out of date and this patch can't be easily merge/applied to it.

          Show
          Elliot Metsger added a comment - Fix committed to 1.1.x branch 609238. The 286 merge branch is out of date and this patch can't be easily merge/applied to it.
          Hide
          Elliot Metsger added a comment -

          See PLUTO-470 for this issue opened against 2.0.0.

          Show
          Elliot Metsger added a comment - See PLUTO-470 for this issue opened against 2.0.0.

            People

            • Assignee:
              Elliot Metsger
              Reporter:
              thanh
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development