Tapestry
  1. Tapestry
  2. TAPESTRY-1595

Add support for ignoring paths that belong to other servlets in the web application

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.0.5
    • Fix Version/s: 5.0.8
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      For pages not handled by Tapestry, it should pass control along to next filter in chain, which it does for paths that do not contain '.' or ':'. For example, if '/foo/bar' is not a page recognized by Tapestry, a URI with the path

      /foo/bar/image.gif

      will cause an 'IllegalArgumentException' with the message 'Unable to resolve page 'foo/bar/image' to a known page name. ...'

      while

      /foo/bar/image_gif

      will be properly forwarded to the filters after TapestryFilter.

      The problem seems to be that 'ComponentActionDispatcher' will attempt to process any path that contains '.' or ':', unlike 'PageRenderDispatcher' which explicitly checks that it recognizes a prefix of the path.

        Issue Links

          Activity

          Mark Thomas made changes -
          Workflow Default workflow, editable Closed status [ 12568440 ] jira [ 12591493 ]
          Mark Thomas made changes -
          Workflow jira [ 12406827 ] Default workflow, editable Closed status [ 12568440 ]
          Howard M. Lewis Ship made changes -
          Resolution Fixed [ 1 ]
          Status In Progress [ 3 ] Closed [ 6 ]
          Fix Version/s 5.0.8 [ 12312898 ]
          Howard M. Lewis Ship made changes -
          Summary TapestryFilter throws IllegalArgumentException if request path for non-Tapestry page contains '.' or ':' Add support for ignoring paths that belong to other servlets in the web application
          Howard M. Lewis Ship made changes -
          Priority Major [ 3 ] Minor [ 4 ]
          Howard M. Lewis Ship made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Howard M. Lewis Ship made changes -
          Assignee Howard M. Lewis Ship [ hlship ]
          Howard M. Lewis Ship made changes -
          Link This issue is duplicated by TAPESTRY-1655 [ TAPESTRY-1655 ]
          Douglas Hauge made changes -
          Field Original Value New Value
          Attachment Tapestry_Exception_Page.html [ 12360341 ]
          Douglas Hauge created issue -

            People

            • Assignee:
              Howard M. Lewis Ship
              Reporter:
              Douglas Hauge
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development