Tapestry 5
  1. Tapestry 5
  2. TAP5-1018

Request to Application Root path ignores ComponentRequestFilter's

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.2, 5.1.0.5
    • Fix Version/s: 5.2.0
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      Example:

      I've made a ComponentRequestFilter to handle Locale issues... And if I request [ /t5app/Start ] page, my Filter code is executed. But, if I request the Application Root path [ /t5app or /t5app/ ] it ignores the Filter and avoids it.

      -------------

      Following [Thiago H. de Paula Figueiredo] comment:

      I've looked at the sources and I've found that RootPathDispatcher, the dispatcher that handles the root URL, doesn't invoke the ComponentEventHandler, which is the pipeline that invokes the ComponentEventFilter's. This is a bug, so please file a JIRA about it.

        Activity

        Hide
        Leonard Lu added a comment -

        This issue could be handled by having the RootPathDispatcher send a redirect rather than handling the request itself.

        For other searchers, a lovely graphic with the Tapestry5 request processing pipelines is available at http://tapestry.formos.com/nightly/tapestry5/images/tapestry_request_processing_800.png, which makes it clear that the current intended behavior is for RootPath to completely bypass the Asset, ComponentEvent and PageRender Dispatchers.

        Show
        Leonard Lu added a comment - This issue could be handled by having the RootPathDispatcher send a redirect rather than handling the request itself. For other searchers, a lovely graphic with the Tapestry5 request processing pipelines is available at http://tapestry.formos.com/nightly/tapestry5/images/tapestry_request_processing_800.png , which makes it clear that the current intended behavior is for RootPath to completely bypass the Asset, ComponentEvent and PageRender Dispatchers.
        Hide
        Ulrich Stärk added a comment -

        You should consider using index pages instead of the deprecated Start page.

        Show
        Ulrich Stärk added a comment - You should consider using index pages instead of the deprecated Start page.

          People

          • Assignee:
            Ulrich Stärk
            Reporter:
            Everton Agner Ramos
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development