Croway opened a new pull request, #1778:
URL: https://github.com/apache/camel-spring-boot/pull/1778
## Summary
- When Spring's async timeout handler sends 503, the Camel executor thread's
`afterProcess()` catches `AsyncRequestNotUsableException` from
`writeResponse()` and falls through to `sendError(500)`, overwriting the 503
- Detect `AsyncRequestNotUsableException` in `afterProcess()` and
`service()` and skip the `sendError(500)` fallback
- Uses class name check to avoid hard compile-time dependency on `spring-web`
## Test plan
- [x] New `SpringBootPlatformHttpAsyncNotUsableTest` with two test methods:
- `testAsyncNotUsableShouldNotOverwriteWith500` — verifies no 500 when
`AsyncRequestNotUsableException` is thrown
- `testRegularIOExceptionStillReturns500` — verifies regular `IOException`
still produces 500
- [x] Test fails before fix (gets 500), passes after fix
- [x] All 99 platform-http tests pass (0 failures, 0 errors) when combined
with #1777
--
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]
