ICEfaces
  1. ICEfaces
  2. ICE-7991

Add ContextPreservingFilter to ICEFaces 1.8-p04 bundle

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-1.8.2.GA_P03
    • Fix Version/s: EE-1.8.2.GA_P04
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      Seam Application with EPS push

      Description

      The original case was after some time with a push application in Seam the link renderer would somehow manage to pull the underlying application name into the rendered link url. It turned out that the request objects were being held by seam filters, and they were being reused by the underlying server, and this caused the context path from subsequent requests to be available (incorrectly) to the link renderer via standard externalContext calls.

      The solution was to provide a filter that held the context-path from the original most request, and to NOT delegate calls to getContextPath to the underlying request object.

      This apparently works, but was provided as a Demo Patch only, and now it's time to include it into the product.

        Activity

        Hide
        Greg Dick added a comment -

        This Filter should be enabled by adding the following to web.xml

        <filter>
        <filter-name>ContextPreservingFilter</filter-name>
        <filter-class>com.icesoft.faces.webapp.http.servlet.ContextPathPreservingFilter</filter-class>
        </filter>

        <filter-mapping>
        <filter-name>ContextPreservingFilter</filter-name>
        <url-pattern>/*</url-pattern>
        </filter-mapping>

        Show
        Greg Dick added a comment - This Filter should be enabled by adding the following to web.xml <filter> <filter-name>ContextPreservingFilter</filter-name> <filter-class>com.icesoft.faces.webapp.http.servlet.ContextPathPreservingFilter</filter-class> </filter> <filter-mapping> <filter-name>ContextPreservingFilter</filter-name> <url-pattern>/*</url-pattern> </filter-mapping>
        Hide
        Greg Dick added a comment -

        Class is checked in

        Show
        Greg Dick added a comment - Class is checked in

          People

          • Assignee:
            Greg Dick
            Reporter:
            Greg Dick
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: