Pluto
  1. Pluto
  2. PLUTO-490

Cannot deploy portlets to ROOT context

    Details

      Description

      Deploying portlets to ROOT fails because of improper detection and handling of the "" context path.

      DefaultApplicationIdResolver#resolveApplicationId(ServletContext) returns "/hostName" for the root context instead of "", where "hostName" is the name of the host containing the context (typically "localhost"). The program will seem to work normally except when HttpServletRequest.getContextPath() is called such as by the JSTL url tag. That method would then return the incorrect value of "/hostName".

      Even when the context path is correctly detected as "", the portletId created by PortletWindowConfig#createPortletId(String, String, String) cannot by parsed by PortletWindowConfig#fromId(String) because the latter throws an exception if the id starts with ".".

      1. pluto_490.diff
        3 kB
        Steven Broadbridge
      2. pluto_rootContext.patch
        5 kB
        Eric Dalquist

        Activity

        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12565322 ] jira [ 12585990 ]
        Mark Thomas made changes -
        Workflow jira [ 12435049 ] Default workflow, editable Closed status [ 12565322 ]
        Eric Dalquist made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Eric Dalquist made changes -
        Attachment pluto_rootContext.patch [ 12386919 ]
        Steven Broadbridge made changes -
        Comment [ Instead of a check and substitute, another possibility is to append "/PlutoInvoker" to whatever is the context path is before calling ServletContext#getContext(String). But the simple check might be more efficient. ]
        Craig Doremus made changes -
        Fix Version/s 2.0.0 [ 12312914 ]
        Fix Version/s 2.0-refactoring [ 12313313 ]
        Eric Dalquist made changes -
        Assignee Eric Dalquist [ edalquist ]
        Fix Version/s 1.1.6 [ 12313075 ]
        Steven Broadbridge made changes -
        Field Original Value New Value
        Attachment pluto_490.diff [ 12385746 ]
        Steven Broadbridge created issue -

          People

          • Assignee:
            Eric Dalquist
            Reporter:
            Steven Broadbridge
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 1h
              1h
              Remaining:
              Remaining Estimate - 1h
              1h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development