[CAMEL-6816] Use exceptionHandler on Xmpp Consumer

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Affects Version/s: 2.12.1
• Component/s: camel-xmpp

Environment

all

• Estimated Complexity: Novice
• Assignee: Unassigned
• Reporter: bounkong khamphousone
• Created: 02/Oct/13 09:14
• Updated: 02/Oct/13 16:57
• Resolved: 02/Oct/13 16:57

Description

Hi!

What do you think about adding a call to the exceptionHandler in the doStart() method of XmppConsumer class ?

It would looks like :

try

{ connection = endpoint.createConnection(); }

catch (XMPPException e) {
if (endpoint.isTestConnectionOnStartup())

{ throw new RuntimeException("Could not connect to XMPP server.", e); }

else

{ final String xmppExceptionLogMessage = XmppEndpoint.getXmppExceptionLogMessage(e); LOG.warn(xmppExceptionLogMessage); getExceptionHandler().handleException(xmppExceptionLogMessage,e); scheduleDelayedStart(); return; }

}

This allowed an exceptionHandler to be notified of failure connection.
The testConnectionOnStartup doesn't call exceptionHandler because the exception is thrown while starting the consumer.