json logger - null value for json attribute if value result is empty

zio · Nov 4, 2023 · 94c1427 · 94c1427
1 parent b118477
commit 94c1427
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 0 deletions.
diff --git a/core/jvm/src/test/scala/zio/logging/JsonLogFormatSpec.scala b/core/jvm/src/test/scala/zio/logging/JsonLogFormatSpec.scala
@@ -298,6 +298,34 @@ object JsonLogFormatSpec extends ZIOSpecDefault {
           val annEscaped = JsonEscape(nonJsonAnnotation)
           assertTrue(result == s"""{"json_annotation":$jsonAnnotation,"nonjson_annotation":"$annEscaped"}""")
         }
+      },
+      test("labeled empty spans") {
+        val format = label("spans", spans)
+        val result = format.toJsonLogger(
+          Trace.empty,
+          FiberId.None,
+          LogLevel.Info,
+          () => "",
+          Cause.empty,
+          FiberRefs.empty,
+          Nil,
+          Map.empty
+        )
+        assertTrue(result == s"""{"spans":null}""")
+      },
+      test("nested labeled empty spans and annotations") {
+        val format = label("data", label("line", line) + label("spans", spans) + label("annotations", allAnnotations))
+        val result = format.toJsonLogger(
+          Trace.empty,
+          FiberId.None,
+          LogLevel.Info,
+          () => "line",
+          Cause.empty,
+          FiberRefs.empty,
+          Nil,
+          Map.empty
+        )
+        assertTrue(result == s"""{"data":{"line":"line","spans":null,"annotations":null}}""")
       }
     )
   )

diff --git a/core/shared/src/main/scala/zio/logging/internal/LogAppender.scala b/core/shared/src/main/scala/zio/logging/internal/LogAppender.scala
@@ -231,6 +231,10 @@ private[logging] object LogAppender {
         result.append("}")
       }
 
+      if (result.isEmpty) {
+        result.append("null")
+      }
+
       stack.pop()
       result
     }