[CAMEL-18119] Regression in 3.4 in date formatting of Simple expression

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.4.0
• Fix Version/s: 3.14.4, 3.18.0
• Component/s: camel-core
• Estimated Complexity: Unknown
• Regression: Regression
• Assignee: Unassigned
• Reporter: Antoine DESSAIGNE
• Created: 17/May/22 11:56
• Updated: 18/May/22 07:23
• Resolved: 17/May/22 19:38

Description

Hello,

We use Joda time and we have custom Joda<>Java converters. For instance, you can convert Joda's ReadableInstant into Java's Date.

Since this commit from CAMEL-15114, Simple expressions can no longer convert header object to a Date object, they can only handle Date and Long types.

I can provide a pull request but I see 2 ways of solving it and I don't know which one you prefer:

1. revert to exchange.getIn().getHeader(key, Date.class) because there's a DateTimeConverter.toDate(Long value) converter if you have a Long
2. keep the check on Date and Long types and then try to convert to Date and then fail if it cannot be converted

Thank you

Issue Links

• links to: GitHub Pull Request #7627