Jetty
  1. Jetty
  2. JETTY-936

DefaultServlet doesn't dispatch to welcome files that are servlets rather than static resources

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 6.1.14
    • Fix Version/s: 6.1.17
    • Component/s: Servlet
    • Labels:
      None
    • Number of attachments :
      1

      Description

      DefaultServlet seems to handle everything about welcome files. However it only checks if the path appended with the welcome file name is a static resource, not if its a handled servlet mapping.

      I discovered this when I compiled all the jsps in an app, including index.jsp, and deleted the source jsps..... the welcome files stopped working.

      One possible solution would be to check the pathMap in ServletHandler for the appended path and dispatch to it if present. However I'm not sure how to get to the ServletHandler from the DefaultServlet. Maybe there's a better way?

      I imagine this is a problem in 7 also but haven't checked.

        Activity

        Athena Yao made changes -
        Field Original Value New Value
        Assignee Athena Yao [ ayao ]
        Athena Yao made changes -
        Attachment jetty936-6 [ 40815 ]
        Jan Bartel made changes -
        Assignee Athena Yao [ ayao ] Greg Wilkins [ gregw ]
        Greg Wilkins made changes -
        Assignee Greg Wilkins [ gregw ] Athena Yao [ ayao ]
        Athena Yao made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 6.1.17.rc1 [ 15220 ]
        Resolution Fixed [ 1 ]
        David Jencks made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Jan Bartel made changes -
        Assignee Athena Yao [ ayao ] Greg Wilkins [ gregw ]
        Greg Wilkins made changes -
        Status Reopened [ 4 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Greg Wilkins
            Reporter:
            David Jencks
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: