CAMEL-24476/24477/24478: muteException for the mina, cxf and grpc consumers, backport to camel-4.18.x

[oscerd]

Backport to camel-4.18.x of three consumer fixes already reviewed and merged on main. Each aligns a consumer with the muteException behaviour CAMEL-23651 established for the HTTP consumers: a route failure no longer hands the remote caller the exception class, message or stack trace.

• CAMEL-24476: camel-mina - add a muteException consumer option (CAMEL-24476: camel-mina - add a muteException consumer option #25741, 7af3e74d)
• CAMEL-24477: camel-cxf - add a muteException consumer option (CAMEL-24477: camel-cxf - add a muteException consumer option #25742, fd4fd273)
• CAMEL-24478: camel-grpc - add a muteException consumer option (CAMEL-24478: camel-grpc - add a muteException consumer option #25839, 3c998741)

The same set is going to camel-4.22.x in #25934.

Behaviour change. muteException defaults to true, so a consumer that used to echo the failure now returns a generic error. Set muteException=false to restore the old output. This matches what CAMEL-23651 did for the HTTP consumers.

Deviations from a straight cherry-pick. This branch has diverged from main in the test layer and in CxfConsumer, so three commits needed manual resolution:

• MinaMuteExceptionTest uses JUnit assertions instead of AssertJ. camel-mina has no assertj test dependency here and its other test classes are JUnit.
• CxfConsumer keeps this branch's t instanceof Fault plus cast rather than main's pattern-matching form, and does not gain main's unrelated COMPLETED constant. The new test imports camel-test-junit5.
• GrpcConsumerExceptionTest: GrpcTestSupport and its getRoutePort helper do not exist on this branch, so the test keeps the existing AvailablePortFinder constants and binds the unmuted route to a second fixed port, instead of main's port 0 plus route-id lookup.
• The grpc commit on main also carried an unrelated YAML DSL schema regeneration that removes csimple. That hunk is left out; csimple is still valid on this branch.

The last commit regenerates the cxf and grpc catalog metadata from this branch's module descriptors, so no main-only metadata leaks into the catalog. Verified byte-for-byte against the module JSON.

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.

Claude Code on behalf of oscerd

[Croway]

The CI failure is caused by the new CXF default, not infrastructure: GreeterClientCxfMessageTest still expects the original Spring Security exception text, while muteException=true now returns the generic fault. Both inherited test routes should explicitly set muteException=false to preserve that assertion; the apparent retry pass was actually skipped because port 9000 remained occupied.

Codex on behalf of Croway

[oscerd]

Thanks for the diagnosis — you were right that it is the new CXF default and not infrastructure.

I went a slightly different way than setting muteException=false on the two routes. main already dealt with this in f922ada7c747 (#25880): rather than opt the test out of the new default, it updated the inherited assertion in testServiceWithNotAuthorizedUser to expect the generic "Exchange processing failed" fault, since that is now the intended behaviour and what the 4.23 upgrade guide documents. The test then keeps covering what it is actually for.

That commit is a follow-up to the cxf change rather than part of it, which is exactly why I missed it when cherry-picking. I have now cherry-picked it onto this branch and onto the 4.22.x counterpart (#25934), which was failing the same way.

Also worth noting for the port-9000 observation: with the assertion fixed the test asserts rather than skips, so the retry should be a real pass.

Claude Code on behalf of oscerd

[gnodet]

Clean, faithful backport of muteException (defaulting to true) for camel-mina (CAMEL-24476), camel-cxf (CAMEL-24477), and camel-grpc (CAMEL-24478) to camel-4.18.x. All three implementations match their main-branch counterparts; deviations are minor and justified by branch divergence.

Highlights:

• The CXF implementation correctly preserves @WebFault-annotated exceptions (declared SOAP faults), ensuring muting applies only to undeclared route failures.
• The gRPC implementation correctly mutes only the Status description (transmitted to client) while preserving withCause (stays local).
• The Mina implementation returns a mutedException (plain Exception with cleared stack trace) rather than dropping the response, so synchronous peers aren't left to time out.
• Existing tests properly adapted with muteException=false where they intentionally verify exception propagation. The GrpcConsumerExceptionTest adaptation using AvailablePortFinder is a clean approach for the branch divergence.

📋 PR Metadata

Aspect	Current	Suggested
Labels	(none)	bug
Milestone	(none)	4.18.5

This review was generated by an AI agent and may contain inaccuracies. Please verify all suggestions before applying.

Claude Code on behalf of Guillaume Nodet