[CAMEL-17563] wireTap doesn't deep copy attachments

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.14.0
• Fix Version/s: 3.16.0
• Component/s: came-core
• Estimated Complexity: Unknown
• Assignee: Claus Ibsen
• Reporter: Yasser Zamani
• Created: 28/Jan/22 13:36
• Updated: 28/Jul/23 11:26
• Resolved: 06/Feb/22 18:02

Description

Attachments are just stored in a Map in an internal property of exchange. So when Camel copies exchange on wireTap, Attachments is copied by reference and so are shared with the main route and so confusing un-deterministic behavior will emerge on wireTap when exchange has attachments!

Currently I've workaround this issue as following:

                    .wireTap("direct:xslt-validation-logging").onPrepare(exchange -> {
                        // deep copy attachments to avoid wiretap added attachments to be seen in the original route
                        AttachmentMessage attMsg = exchange.getIn(AttachmentMessage.class);
                        attMsg.setAttachments(attMsg.getAttachments());
                    }).end()

but I think Camel internally MUST do this for me as it's useless to share attachments with main route I think.

Issue Links

• is related to: CAMEL-19088 camel-core - Remove SafeCopyProperty that was only in use for camel-zipkin
• relates to: CAMEL-19675 Attachment removed from Exchange after AggregationStrategy