fix(logger): reduce severity of log failure

Matthew Fisher · Matthew Fisher · commit 4f7ad8a6e572 · 2015-06-26T16:07:54.000-07:00
Sending a log to an external server should not kill the logger on an error. For example, if the host is down or unavailable, the logger should just log that the error message has been lost, but carry on.
diff --git a/logger/drain/drain.go b/logger/drain/drain.go
@@ -26,7 +26,7 @@ func SendToDrain(m string, drain string) error {
 func sendToSyslogDrain(m string, drain string) error {
 	conn, err := net.Dial("udp", drain)
 	if err != nil {
-		log.Fatal(err)
+		log.Print(err)
 	}
 	defer conn.Close()
 	fmt.Fprintf(conn, m)

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ func SendToDrain(m string, drain string) error {`
`26`	`26`	`func sendToSyslogDrain(m string, drain string) error {`
`27`	`27`	`conn, err := net.Dial("udp", drain)`
`28`	`28`	`if err != nil {`
`29`		`- log.Fatal(err)`
	`29`	`+ log.Print(err)`
`30`	`30`	`}`
`31`	`31`	`defer conn.Close()`
`32`	`32`	`fmt.Fprintf(conn, m)`