Added BugSnag error tracker

broker/src/main/java/io/moquette/BrokerConstants.java

@@ -61,6 +61,10 @@ public final class BrokerConstants {
     public static final String METRICS_LIBRATO_EMAIL_PROPERTY_NAME = "metrics.librato.email";
     public static final String METRICS_LIBRATO_TOKEN_PROPERTY_NAME = "metrics.librato.token";
     public static final String METRICS_LIBRATO_SOURCE_PROPERTY_NAME = "metrics.librato.source";
+
+    public static final String BUGSNAG_ENABLE_PROPERTY_NAME = "use_bugsnag";
+    public static final String BUGSNAG_TOKEN_PROPERTY_NAME = "bugsnag.token";
+
     public static final String STORAGE_CLASS_NAME = "storage_class";
 
     private BrokerConstants() {

broker/src/main/java/io/moquette/server/netty/BugSnagErrorsHandler.java

@@ -0,0 +1,39 @@
+/*
+ * Copyright (c) 2012-2017 The original author or authors
+ * ------------------------------------------------------
+ * All rights reserved. This program and the accompanying materials
+ * are made available under the terms of the Eclipse Public License v1.0
+ * and Apache License v2.0 which accompanies this distribution.
+ *
+ * The Eclipse Public License is available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ *
+ * The Apache License v2.0 is available at
+ * http://www.opensource.org/licenses/apache2.0.php
+ *
+ * You may elect to redistribute this code under either of these licenses.
+ */
+package io.moquette.server.netty;
+
+import com.bugsnag.Bugsnag;
+import io.moquette.server.config.IConfig;
+import io.netty.channel.ChannelHandlerContext;
+import io.netty.channel.ChannelInboundHandlerAdapter;
+
+import static io.moquette.BrokerConstants.BUGSNAG_TOKEN_PROPERTY_NAME;
+
+public class BugSnagErrorsHandler extends ChannelInboundHandlerAdapter {
+
+    private Bugsnag bugsnag;
+
+    public void init(IConfig props) {
+        final String token = props.getProperty(BUGSNAG_TOKEN_PROPERTY_NAME);
+        this.bugsnag = new Bugsnag(token);
+    }
+
+    @Override
+    public void exceptionCaught(ChannelHandlerContext ctx, Throwable cause) throws Exception {
+        bugsnag.notify(cause);
+        ctx.fireExceptionCaught(cause);
+    }
+}

broker/src/main/java/io/moquette/server/netty/NettyAcceptor.java

@@ -97,6 +97,7 @@ public class NettyAcceptor implements ServerAcceptor {
     BytesMetricsCollector m_bytesMetricsCollector = new BytesMetricsCollector();
     MessageMetricsCollector m_metricsCollector = new MessageMetricsCollector();
     private Optional<? extends ChannelInboundHandler> metrics;
+    private Optional<? extends ChannelInboundHandler> errorsCather;
 
     private int nettySoBacklog;
     private boolean nettySoReuseaddr;
@@ -145,6 +146,14 @@ public class NettyAcceptor implements ServerAcceptor {
             this.metrics = Optional.empty();
         }
 
+        final boolean useBugSnag = Boolean.parseBoolean(props.getProperty(BUGSNAG_ENABLE_PROPERTY_NAME, "false"));
+        if (useBugSnag) {
+            BugSnagErrorsHandler bugSnagHandler = new BugSnagErrorsHandler();
+            bugSnagHandler.init(props);
+            this.errorsCather = Optional.of(bugSnagHandler);
+        } else {
+            this.errorsCather = Optional.empty();
+        }
         initializePlainTCPTransport(mqttHandler, props);
         initializeWebSocketTransport(mqttHandler, props);
         String sslTcpPortProp = props.getProperty(BrokerConstants.SSL_PORT_PROPERTY_NAME);