gnodet opened a new pull request, #2955:
URL: https://github.com/apache/cxf/pull/2955
## Summary
- **Removes dead pre-Servlet 3.1 backward-compatibility code** — CXF now
requires Jakarta Servlet 6.x, so the runtime reflection checks for Servlet
3.0/3.1 availability are unreachable dead code
- Removes `isServlet3` field and
`ServletRequest.class.getMethod("isAsyncSupported")` reflection check from
`AbstractHTTPDestination`
- Removes `IS_31` static check and `Servlet31Continuation` subclass from
`Servlet3ContinuationProvider`, merging the Servlet 3.1 behavior (NIO
`WriteListener` support, `isReady()`) into the base `Servlet3Continuation` class
- Simplifies `retrieveFromContinuation()` and `setupContinuation()` methods
- **Fixes `FormUtils.isFormPostRequest()`** to use `startsWith` instead of
`equals` for content-type matching, so content types with parameters (e.g.,
`application/x-www-form-urlencoded;charset=UTF-16BE`) are correctly recognized
as form POST requests
Supersedes #702 (servlet cleanup) and the `FormUtils` fix from #698
(CXF-8347).
## Test plan
- [x] `rt/transports/http` module compiles and tests pass
- [x] `rt/frontend/jaxrs` module compiles and tests pass
- [x] `AsyncResponseImplTest` passes (uses `Servlet3ContinuationProvider`)
- [ ] CI pipeline
🤖 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]
