fix: Switch default logging to JSON

Signed-off-by: Teddy Wahle <[email protected]>
argoproj · Jan 24, 2025 · 662dcf7 · 662dcf7
1 parent bcf2143
commit 662dcf7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/util/log/logrus.go b/util/log/logrus.go
@@ -46,8 +46,7 @@ func CreateFormatter(logFormat string) logrus.Formatter {
 			TimestampFormat: checkTimestampFormat(),
 		}
 	default:
-		formatType = &logrus.TextFormatter{
-			FullTimestamp:   checkEnableFullTimestamp(),
+		formatType = &logrus.JSONFormatter{
 			TimestampFormat: checkTimestampFormat(),
 		}
 	}

diff --git a/util/log/logrus_test.go b/util/log/logrus_test.go
@@ -48,6 +48,6 @@ func TestCreateFormatter(t *testing.T) {
 	})
 	t.Run("log format is not json or text", func(t *testing.T) {
 		result := CreateFormatter("xml")
-		assert.Equal(t, &logrus.TextFormatter{}, result)
+		assert.Equal(t, &logrus.JSONFormatter{}, result)
 	})
 }