Use plain logger as default

config.dev.toml

@@ -81,7 +81,7 @@ LogLevel = "debug"
 # LogFormat determines the format that logs are output in. Defaults to json.
 # It is only used when CloudRuntime is set to none. Values can be "plain" or "json".
 # Env override: ATHENS_LOG_FORMAT
-LogFormat = "json"
+LogFormat = "plain"
 
 # CloudRuntime is the Cloud Provider on which the Proxy/Registry is running.
 # Currently available options are "GCP", or "none". Defaults to "none"

pkg/config/config.go

@@ -151,7 +151,7 @@ func defaultConfig() *Config {
 		GoGetWorkers:     10,
 		ProtocolWorkers:  30,
 		LogLevel:         "debug",
-		LogFormat:        "json",
+		LogFormat:        "plain",
 		CloudRuntime:     "none",
 		EnablePprof:      false,
 		PprofPort:        ":3001",

pkg/config/config_test.go

@@ -258,7 +258,7 @@ func TestParseExampleConfig(t *testing.T) {
 	expConf := &Config{
 		GoEnv:           "development",
 		LogLevel:        "debug",
-		LogFormat:       "json",
+		LogFormat:       "plain",
 		GoBinary:        "go",
 		GoGetWorkers:    10,
 		ProtocolWorkers: 30,

pkg/log/log_test.go

@@ -97,6 +97,18 @@ var testCases = []input{
 	},
 	{
 		name:          "default",
+		cloudProvider: "none",
+		level:         logrus.DebugLevel,
+		fields:        logrus.Fields{"xyz": "abc", "abc": "xyz"},
+		logFunc: func(e Entry) time.Time {
+			t := time.Now()
+			e.Warnf("warn message")
+			return t
+		},
+		output: `WARNING[%v]: warn message` + "\t" + `abc=xyz xyz=abc` + " \n",
+	},
+	{
+		name:          "default json",
 		format:        "json",
 		cloudProvider: "none",
 		level:         logrus.DebugLevel,
@@ -121,7 +133,7 @@ func TestCloudLogger(t *testing.T) {
 			out := buf.String()
 			expected := tc.output
 			if strings.Contains(expected, "%v") {
-				if tc.format == "plain" {
+				if tc.format == "plain" || (tc.format == "" && (tc.cloudProvider == "none" || tc.cloudProvider == "")) {
 					expected = fmt.Sprintf(expected, entryTime.Format(time.Kitchen))
 				} else {
 					expected = fmt.Sprintf(expected, entryTime.Format(time.RFC3339))