[jira] [Created] (TAP5-2074) Extra leading slashes in URLs in HTTP 404 error pages

["Dmitry Gusev (JIRA)" <ji...@apache.org>]

Dmitry Gusev created TAP5-2074:
----------------------------------

             Summary: Extra leading slashes in URLs in HTTP 404 error pages
                 Key: TAP5-2074
                 URL: https://issues.apache.org/jira/browse/TAP5-2074
             Project: Tapestry 5
          Issue Type: Bug
          Components: tapestry-core
    Affects Versions: 5.3.6
            Reporter: Dmitry Gusev


When implementing custom HTTP error page in Tapestry5 (like HTTP 404 error page described here: http://tapestry.apache.org/error-page-recipe.html), all client URLs contain extra leading slashes, which makes those links invalid.

By client URLs I mean URLs generated by tapestry for t:pagelink and all the assets (js, css, images, etc.).

I had this bug in Tapestry 5.3.6 application deployed at Tomcat 7.0.29 as the ROOT context.

The problem seems to be in HttpServletRequest.getContextPath() which returns "/" when request has been handled by ERROR dispatcher.

Here's the javadoc of org.apache.tapestry5.services.Request.getContextPath():

/**
     * Returns the context path. This always starts with a "/" character and does not end with one, with the exception
     * of servlets in the root context, which return the empty string.
     */
    String getContextPath();

Default implementation of Tapestry5's RequestImpl.getContextPath() is simply delegates contextPath retrieval to servlet API:

    public String getContextPath()
    {
        return request.getContextPath();
    }

Not sure if this is Tomcat or Tapestry bug, but Tapestry should definitely handle this situation.

As a temporary workaround I had to implement MethodAdvice for Request.getContextPath() in my AppModule like this:


    @Match("Request")
    public void adviseRequestContextPath(MethodAdviceReceiver receiver) throws SecurityException, NoSuchMethodException
    {
        if (!receiver.getInterface().equals(Request.class))
        {
            return;
        }
        
        @SuppressWarnings("unchecked")
        Method method = receiver.getInterface().getDeclaredMethod("getContextPath");
        
        receiver.adviseMethod(method, new MethodAdvice()
        {
            @Override
            public void advise(MethodInvocation invocation)
            {
                MethodInvocation proceed = invocation.proceed();
                
                if ("/".equals(proceed.getReturnValue()))
                {
                    proceed.setReturnValue("");
                }
            }
        });
    }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira