CloudJsonLoggingAppender: fix illegal characters in log names

nafg · Jan 30, 2019 · 92669ba · 92669ba
1 parent e269256
commit 92669ba
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/appender/src/main/scala/io/github/nafg/cloudlogging/appender/CloudJsonLoggingAppender.scala b/appender/src/main/scala/io/github/nafg/cloudlogging/appender/CloudJsonLoggingAppender.scala
@@ -78,6 +78,9 @@ object CloudJsonLoggingAppender {
       m.getName
   }
 
+  val legalNameChars = ('a' to 'z').toSet ++ ('A' to 'Z') ++ ('0' to '9') ++ Set('/', '_', '-', '.')
+  def legalizeNameChar(c: Char) = if (legalNameChars(c)) c else '-'
+
   def logEntryFor(e: ILoggingEvent) = {
     val level = e.getLevel
 
@@ -133,7 +136,7 @@ object CloudJsonLoggingAppender {
 
     LogEntry
       .newBuilder(Payload.JsonPayload.of(payload.asJava))
-      .setLogName(e.getLoggerName)
+      .setLogName(e.getLoggerName.map(legalizeNameChar))
       .setTimestamp(e.getTimeStamp)
       .setSeverity(severityFor(level))
       .setLabels(labels.asJava)