Pluto
  1. Pluto
  2. PLUTO-593

Will not load portlets deployed in root servlet context

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.1
    • Fix Version/s: 2.0.2, 2.1.0
    • Component/s: portal driver
    • Labels:
      None

      Description

      Pluto does contextPath.substring(1) on the value returned by ServletContext.getContextPath() Per the Servlet spec that method will return an empty string if the servlet is deployed in the root context, this will result in an IndexOutOfBounds exception.

      The proposed fix is to not remove the leading /.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        2d 6h 41m 1 Eric Dalquist 14/May/10 01:20
        Resolved Resolved Reopened Reopened
        12d 19h 19m 1 Ate Douma 26/May/10 20:39
        Reopened Reopened Resolved Resolved
        2m 25s 1 Ate Douma 26/May/10 20:41
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12565138 ] jira [ 12585868 ]
        Mark Thomas made changes -
        Workflow jira [ 12510640 ] Default workflow, editable Closed status [ 12565138 ]
        Ate Douma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Ate Douma added a comment -

        Fix applied

        Show
        Ate Douma added a comment - Fix applied
        Ate Douma made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Assignee Eric Dalquist [ edalquist ] Ate Douma [ adouma ]
        Hide
        Ate Douma added a comment -

        o.a.p.pluto-portal-driver-impl.EventProviderImpl.isDeclaredAsPublishingEvent method is still stripping leading '/' from the context path to determine the application name, causing event handling to fail.

        Show
        Ate Douma added a comment - o.a.p.pluto-portal-driver-impl.EventProviderImpl.isDeclaredAsPublishingEvent method is still stripping leading '/' from the context path to determine the application name, causing event handling to fail.
        Eric Dalquist made changes -
        Fix Version/s 2.0.2 [ 12314872 ]
        Eric Dalquist made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Eric Dalquist added a comment -

        Fixed by not removing / from the context path when generating the application name.

        Show
        Eric Dalquist added a comment - Fixed by not removing / from the context path when generating the application name.
        Eric Dalquist made changes -
        Fix Version/s 2.1.0 [ 12315040 ]
        Fix Version/s 2.0.2 [ 12314872 ]
        Eric Dalquist made changes -
        Field Original Value New Value
        Assignee Eric Dalquist [ edalquist ]
        Eric Dalquist created issue -

          People

          • Assignee:
            Ate Douma
            Reporter:
            Eric Dalquist
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development