paulrutter opened a new pull request, #582:
URL: https://github.com/apache/felix-dev/pull/582

   Fixes [FELIX-6873](https://issues.apache.org/jira/browse/FELIX-6873).
   
   ERROR dispatch matched URL-pattern filters against the original request URI 
instead of the path of the selected error servlet. This is inconsistent with 
FORWARD/INCLUDE and with servlet containers.
   
   **Change**
   - `ServletResponseWrapper` now matches ERROR filters against the error 
servlet's own patterns (`/foo/*` becomes `/foo`, `/` and empty become `/`).
   - Extension patterns have no path to match; an error servlet without 
patterns is matched by servlet-name filters only.
   - `FilterRegistry` / `PerContextHandlerRegistry` get a 
`getFilterHandlers(..., String[])` overload; the existing single-URI method 
delegates to it.
   
   **Behaviour change:** ERROR filters that relied on matching the original 
request URI no longer match. Worth a release note.
   
   **Tests:** new `FilterRegistryTest.testErrorFilterMatchesErrorServletPath` 
and `ServletResponseWrapperTest`; full `http/base` suite passes (147 tests, JDK 
21).
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to