[CAMEL-4117] RedeliverErrorHandler - Should make defensive copy of Exchange so redelivery works for using Processor

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.7.0
• Fix Version/s: 2.8.0
• Component/s: camel-core
• Assignee: Claus Ibsen
• Reporter: Claus Ibsen
• Created: 18/Jun/11 15:54
• Updated: 20/Jun/11 10:49
• Resolved: 20/Jun/11 10:49

Description

When using redelivery with the Camel error handler, it will by default use the previous exchange when redelivering. This ensures that any changes that was done by the failed attempt does not reflect when a redelivery is performed (eg think like use original message). However this does not apply when end users uses the low level .process in their Camel routes. So if they change the Exchange in the .process and then some error occurs, then that change is redelivered.

We should consider making the redelivery error handler do a defensive copy of the exchange, so it can redelivery without any changes/side effects. For example this happens when people use .bean or any of the other EIPs. So why not for .process as well?

Issue Links

• is related to: CAMEL-1817 Moved logic for defensive copy of current exchange from Pipeline to RedeliveryErrorHandler