Implement single error handler, start moving functions that discarded errors over to it

error.go

@@ -0,0 +1,22 @@
+package main
+
+import (
+	"encoding/json"
+	"fmt"
+	"net/http"
+)
+
+type ErrorBody struct {
+	Error string
+}
+
+func writeHTTPError(res http.ResponseWriter, err error) {
+	res.WriteHeader(400)
+	encodedOutput, err := json.Marshal(ErrorBody{Error: err.Error()})
+	if err != nil {
+		fmt.Println("Failed to generate HTTP error: " + err.Error())
+	} else {
+		res.Write(encodedOutput)
+	}
+
+}

fail2rest.go

@@ -15,10 +15,6 @@ type Configuration struct {
 	Fail2banSocket string
 }
 
-type ErrorBody struct {
-	Error string
-}
-
 var fail2goConn *fail2go.Conn
 
 func main() {

global.go

@@ -8,10 +8,17 @@ import (
 )
 
 func globalStatusHandler(res http.ResponseWriter, req *http.Request, fail2goConn *fail2go.Conn) {
-	globalStatus, _ := fail2goConn.GlobalStatus()
+	globalStatus, err := fail2goConn.GlobalStatus()
 
-	encodedOutput, err := json.Marshal(globalStatus)
 	if err != nil {
+		writeHTTPError(res, err)
+		return
+	}
+	encodedOutput, err := json.Marshal(globalStatus)
+
+	if err != nil {
+		writeHTTPError(res, err)
+		return
 	}
 
 	res.Write(encodedOutput)