[CAMEL-9119] XSLT errors should not be ignored

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.14.3
• Fix Version/s: 2.16.0, 2.15.4
• Component/s: camel-core

Environment

ServiceMix 5.4.1

• Patch Info: Patch Available
• Estimated Complexity: Unknown
• Assignee: Claus Ibsen
• Reporter: metatech
• Created: 07/Sep/15 13:33
• Updated: 16/Sep/15 13:02
• Resolved: 16/Sep/15 13:02

Description

Since CAMEL-4396, XSLT exceptions are not logged anymore on System.err.
Unfortunately, the XsltErrorHandler which was introduced does not behave as the default ErrorHandler which logs on System.err and re-throws exceptions (my mistake).
The XsltErrorHandler should re-throw exceptions for ERROR or FATAL, otherwise they are ignored and the service is allowed to start, although the XSLT transformation is not successfully started.
Here is a small patch that fixes that.
The test "XsltTestErrorListenerTest" is still successful.