CAMEL-24476/24477/24478: muteException for the mina, cxf and grpc consumers, backport to camel-4.22.x - #25934
Open
oscerd wants to merge 4 commits into
Open
Conversation
The consumer wrote exchange.getException() straight back over the socket, so a route failure handed the remote peer the exception class and message over a textline codec, and its serialised form over the object codec. muteException defaults to true, matching the http consumers aligned by CAMEL-23651. (cherry picked from commit 7af3e74) MinaMuteExceptionTest uses JUnit assertions here: camel-mina on this branch has no assertj test dependency, and its other 36 test classes are JUnit. Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
An exception thrown by the route was described back to the SOAP client in the fault, exposing internal types and messages. Declared faults (@webfault) are still reported as before, since those are part of the service contract. (cherry picked from commit fd4fd27) Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
The consumer put the route exception's message into the gRPC status description, so the caller received internal detail on failure. (cherry picked from commit 3c99874) The main commit also carried an unrelated YAML DSL schema regeneration that drops csimple; that hunk is left out, csimple is still valid on this branch. Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Contributor
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
davsclaus
approved these changes
Aug 31, 2026
Contributor
|
🧪 CI tested the following changed modules:
🔬 Scalpel shadow comparison — Scalpel: 19 tested, 26 compile-only — current: 16 all testedMaveniverse Scalpel detected 45 affected modules (current approach: 16).
|
Croway
approved these changes
Aug 31, 2026
CxfEndpoint's muteException consumer option now defaults to true (CAMEL-24477), so an undeclared route failure such as this test's authorization denial no longer leaks its message to the SOAP caller. (cherry picked from commit f922ada) Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Backport to
camel-4.22.xof three consumer fixes already reviewed and merged onmain. Each aligns a consumer with themuteExceptionbehaviour CAMEL-23651 established for the HTTP consumers: a route failure no longer hands the remote caller the exception class, message or stack trace.muteExceptionconsumer option (CAMEL-24476: camel-mina - add a muteException consumer option #25741,7af3e74d)muteExceptionconsumer option (CAMEL-24477: camel-cxf - add a muteException consumer option #25742,fd4fd273)muteExceptionconsumer option (CAMEL-24478: camel-grpc - add a muteException consumer option #25839,3c998741)Behaviour change.
muteExceptiondefaults totrue, so a consumer that used to echo the failure now returns a generic error. SetmuteException=falseto restore the old output. This matches what CAMEL-23651 did for the HTTP consumers.Deviations from a straight cherry-pick, both mechanical:
MinaMuteExceptionTestuses JUnit assertions instead of AssertJ.camel-minahas no assertj test dependency on this branch and its other 36 test classes are JUnit, so adding one was the wrong trade for a backport.maincarried an unrelated YAML DSL schema regeneration that removescsimplefrom the expression list. That hunk is left out;csimpleis still valid on this branch.The upgrade-guide entries are not included: the guides for every line live on
main.Built and tested per module on this branch (
camel-mina,camel-cxf-soap,camel-grpc), including the new and touched tests. Catalog and DSL mirrors regenerated from this branch, so nomain-only metadata leaks in.Claude Code on behalf of oscerd