From 360c0c35f5ba4605dec8afd8dca8768959f06f0a Mon Sep 17 00:00:00 2001
From: "Michael M. Wasser" <untoldone@users.noreply.github.com>
Date: Tue, 29 Aug 2017 13:02:31 -0700
Subject: [PATCH] remove default stats logger to prevent error in docker

---
 api/log.go                    | 6 +++++-
 config.toml.sample            | 2 +-
 middleware/record_features.go | 3 ++-
 3 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/api/log.go b/api/log.go
index db0c3a0..9815a1e 100644
--- a/api/log.go
+++ b/api/log.go
@@ -18,7 +18,11 @@ func StatsLogger() *log.Logger {
 		logStyle := viper.GetString("statsLogger")
 		switch logStyle {
 		case "syslog":
-			statsWriter, _ = syslog.New(syslog.LOG_NOTICE, "bloomapi-stats")
+			var err error
+			statsWriter, err = syslog.New(syslog.LOG_NOTICE, "bloomapi-stats")
+			if err != nil {
+				log.Fatal("Error sending logs to syslog, check BloomAPI configuration")
+			}
 		case "stdout":
 			statsWriter = os.Stdout
 		default:
diff --git a/config.toml.sample b/config.toml.sample
index 7dd4cf1..035e57c 100644
--- a/config.toml.sample
+++ b/config.toml.sample
@@ -4,5 +4,5 @@ searchHosts = [
 ]
 bloomapiPort = 3005
 yourchartUrl = "http://yourchart:3001"
-statsLogger = "syslog" # values can be: 'syslog' or 'stdout'
 generalLogger = "stdout"
+#statsLogger = "syslog" # values can be: 'syslog' or 'stdout', default is nothing
diff --git a/middleware/record_features.go b/middleware/record_features.go
index 0538c95..125df93 100644
--- a/middleware/record_features.go
+++ b/middleware/record_features.go
@@ -17,6 +17,7 @@ func (s *RecordFeatures) ServeHTTP(rw http.ResponseWriter, r *http.Request, next
 	features := api.GetFeatures(r)
 	j, _ := json.Marshal(features)
 	featureJson := string(j)
-	api.StatsLogger().Println(featureJson)
+	stats := api.StatsLogger()
+	stats.Println(featureJson)
 	next(rw, r)
 }
\ No newline at end of file