guard on the time series point before logging

JonathanLennox · Jun 14, 2019 · 1ae7142 · 1ae7142
1 parent f63526a
commit 1ae7142
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/src/main/java/org/jitsi/utils/logging/TimeSeriesLogger.java b/src/main/java/org/jitsi/utils/logging/TimeSeriesLogger.java
@@ -97,7 +97,10 @@ public boolean isInfoEnabled()
      */
     public void trace(Map<String, Object> point)
     {
-        logger.trace(new JSONObject(point).toJSONString());
+        if (point != null && !point.isEmpty())
+        {
+            logger.trace(new JSONObject(point).toJSONString());
+        }
     }
 
     /**
@@ -107,7 +110,10 @@ public void trace(Map<String, Object> point)
      */
     public void warn(Map<String, Object> point)
     {
-        logger.warn(new JSONObject(point).toJSONString());
+        if (point != null && !point.isEmpty())
+        {
+            logger.warn(new JSONObject(point).toJSONString());
+        }
     }
 
     /**
@@ -117,6 +123,9 @@ public void warn(Map<String, Object> point)
      */
     public void info(Map<String, Object> point)
     {
-        logger.info(new JSONObject(point).toJSONString());
+        if (point != null && !point.isEmpty())
+        {
+            logger.info(new JSONObject(point).toJSONString());
+        }
     }
 }