ammachado commented on PR #23913: URL: https://github.com/apache/camel/pull/23913#issuecomment-4674780976
The `muteException=true` default change in this PR broke `VertxHttpTransferExceptionTest` in `camel-vertx-http`. That test uses an Undertow consumer as the server-side endpoint with `transferException=true`, but now `muteException=true` takes precedence and suppresses exception serialization, so the client receives an empty `text/plain` 500 response instead of the serialized `IllegalStateException`. The fix is to add `muteException=false` to the Undertow consumer URI in that test, consistent with the changes made here to `UndertowTransferExceptionTest`. It is a one-line change: ```java // components/camel-vertx/camel-vertx-http/src/test/java/org/apache/camel/component/vertx/http/VertxHttpTransferExceptionTest.java from(getTestServerUri() + "?transferException=true&muteException=false") ``` Worth checking whether any other cross-component tests that use an Undertow (or netty-http) consumer with `transferException=true` need the same opt-out. _Claude Code on behalf of Adriano Machado_ -- 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]
