Merge pull request #526 from michael-loki/context_json_metrics

Make JSON metrics opt-in
5 years ago · f144c25620
parent aeef015d80 fb0f08ec01
commit f144c25620
1 changed files with 7 additions and 4 deletions
--- a/llarp/context.cpp
+++ b/llarp/context.cpp
@ -146,10 +146,13 @@ namespace llarp
          *m_scheduler, m_metricsManager->instance());
    }

-    m_metricsManager->instance()->addGlobalPublisher(
-        std::make_shared< metrics::JsonPublisher >(
-            std::bind(&metrics::JsonPublisher::directoryPublisher,
-                      std::placeholders::_1, jsonMetricsPath)));
+    if(!jsonMetricsPath.native().empty())
+    {
+      m_metricsManager->instance()->addGlobalPublisher(
+          std::make_shared< metrics::JsonPublisher >(
+              std::bind(&metrics::JsonPublisher::directoryPublisher,
+                        std::placeholders::_1, jsonMetricsPath)));
+    }

    if(!metricTankHost.empty())
    {