Pax Web

Welcome files do not follow servlet specs

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: Web Container
  • Labels:
    None

Description

Pax Web does not implement handling of welcome files according to servlet specs (e.g. section SRV.9.10 of servlet specs 2.4).
Pax Web will forward/redirect to a configured welcome file in the moment that the url ends with an "/" and the welcome file is found (relative to the requested path) while the specs mandates that the welcome file should be should in case that "request for a URI that corresponds to a directory entry in the WAR not mapped to a Web component."

So, Pax Web should only look for an welcome file if the request was NOT handled by another web component and is an directory entry (ends with "/"?)

More, the current implementation, which is based on a filter, cannot be used as filters are not applied if the request does not match a servlet for the request.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Achim Nierbeck added a comment -

looking at the code unsure if this one is still open?

Show
Achim Nierbeck added a comment - looking at the code unsure if this one is still open?
Hide
Permalink
Achim Nierbeck added a comment -

let's try to fix this, or at least the filter chaining so that the example for PAXWEB-154 is able to work

Show
Achim Nierbeck added a comment - let's try to fix this, or at least the filter chaining so that the example for PAXWEB-154 is able to work
Hide
Permalink
Achim Nierbeck added a comment -

This could be a rather large refactoring regarding of how welcome files work in pax web and might even effect the default servlet.
Therefore this won't be in the 2.0.0 version

Show
Achim Nierbeck added a comment - This could be a rather large refactoring regarding of how welcome files work in pax web and might even effect the default servlet. Therefore this won't be in the 2.0.0 version

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated: