Dont start reporters when keys are missing

billoneil · billoneil · commit 434704fdd468 · 2019-02-26T20:55:34.000-05:00
diff --git a/stubbornjava-common/src/main/java/com/stubbornjava/common/MetricsReporters.java b/stubbornjava-common/src/main/java/com/stubbornjava/common/MetricsReporters.java
@@ -2,6 +2,9 @@
 
 import java.util.concurrent.TimeUnit;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import com.codahale.metrics.MetricFilter;
 import com.codahale.metrics.MetricRegistry;
 import com.codahale.metrics.graphite.GraphiteReporter;
@@ -10,13 +13,20 @@
 
 // {{start:reporters}}
 class MetricsReporters {
+    private static final Logger log = LoggerFactory.getLogger(MetricsReporters.class);
 
     public static void startReporters(MetricRegistry registry) {
         // Graphite reporter to Grafana Cloud
         OkHttpClient client = new OkHttpClient.Builder()
             //.addNetworkInterceptor(HttpClient.getLoggingInterceptor())
             .build();
 
+        if (!Configs.properties().hasPath("metrics.graphite.host")
+            || !Configs.properties().hasPath("metrics.grafana.api_key")) {
+            log.info("Missing metrics reporter key or host skipping");
+            return;
+        }
+
         String graphiteHost = Configs.properties().getString("metrics.graphite.host");
         String grafanaApiKey = Configs.properties().getString("metrics.grafana.api_key");
         final GraphiteHttpSender graphite = new GraphiteHttpSender(client, graphiteHost, grafanaApiKey);
