Jetspeed 2
  1. Jetspeed 2
  2. JS2-681

Login portlet doesn't run in Tomcat ROOT context

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.1.2
    • Component/s: Admin Portlets
    • Labels:
      None
    • Environment:
      JDK 1.6, Tomcat 5.5, Linux Ubuntu 6.10

      Description

      We have changed the application context of a Jetspeed 2 portal from "/$

      {artifactId}

      " to "/". Doing that, the Jetspeed portal runs perfectly but the Login portlet fails. The portal throws the following exception:

      javax.portlet.PortletException
      at org.apache.jetspeed.dispatcher.JetspeedRequestDispatcher.include(JetspeedRequestDispatcher.java:102)
      at org.apache.portals.bridges.common.GenericServletPortlet.doView(GenericServletPortlet.java:363)
      at javax.portlet.GenericPortlet.doDispatch(GenericPortlet.java:247)
      ......
      Caused by: javax.servlet.ServletException: In URL tags, when the "context" attribute is specified, values of both "context" and "url" must start with "/".

      The problem is that the "context" attribute of the "<c:url />" tag cannot have an empty value. The empty value is returned by the "$

      {requestContext.request.contextPath}

      " expression (see the Servlet API::HttpServletRequest.getContextPath() to know the reason).

      We have attached a patch that fix this bug. In addition to this patch, the JSTL API and implementation must be upgraded to jstl-1.1.1.jar and standard-1.1.2.jar.

      Regards.

      1. patch-login-portlet.txt
        6 kB
        Enrique Ruiz (DiSiD Technologies S.L.)

        Activity

        Hide
        Ate Douma added a comment -

        Should be properly fixed now

        Show
        Ate Douma added a comment - Should be properly fixed now
        Hide
        Ate Douma added a comment -

        While testing with Jetty, it turns out these fixes create invalid urls like /jetspeed//login/proxy (notice the double slashes).

        Show
        Ate Douma added a comment - While testing with Jetty, it turns out these fixes create invalid urls like /jetspeed//login/proxy (notice the double slashes).
        Hide
        Ate Douma added a comment -

        Patch applied and upgrade jstl and standard libs to version 1.1.2
        Jetspeed working nicely now deployed under ROOT

        Show
        Ate Douma added a comment - Patch applied and upgrade jstl and standard libs to version 1.1.2 Jetspeed working nicely now deployed under ROOT

          People

          • Assignee:
            Ate Douma
            Reporter:
            Enrique Ruiz (DiSiD Technologies S.L.)
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development