camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "bounkong khamphousone (JIRA)" <j...@apache.org>
Subject [jira] [Created] (CAMEL-6814) use exceptionHandler on XmppConsumer
Date Wed, 02 Oct 2013 08:57:23 GMT
bounkong khamphousone created CAMEL-6814:
--------------------------------------------

             Summary: use exceptionHandler on XmppConsumer
                 Key: CAMEL-6814
                 URL: https://issues.apache.org/jira/browse/CAMEL-6814
             Project: Camel
          Issue Type: Improvement
          Components: camel-xmpp
            Reporter: bounkong khamphousone
            Priority: Trivial


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.




--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message