diff options
author | Ben Morrison <ben@gbmor.dev> | 2019-06-10 00:47:30 -0400 |
---|---|---|
committer | Ben Morrison <ben@gbmor.dev> | 2019-06-10 00:49:34 -0400 |
commit | 824556ab9829e0ff369808621bacb5cccbee27e9 (patch) | |
tree | 13e5dcf9cbfeec4763708daac244b7e30cd3c525 /svc/http.go | |
parent | bedd5d588f4375b486734026c5eda861ab2bf485 (diff) | |
download | getwtxt-824556ab9829e0ff369808621bacb5cccbee27e9.tar.gz |
refactored http err handling to combine 3 funcs into 1
Diffstat (limited to 'svc/http.go')
-rw-r--r-- | svc/http.go | 21 |
1 files changed, 4 insertions, 17 deletions
diff --git a/svc/http.go b/svc/http.go index 347430e..0118912 100644 --- a/svc/http.go +++ b/svc/http.go @@ -2,6 +2,7 @@ package svc // import "github.com/getwtxt/getwtxt/svc" import ( "context" + "fmt" "log" "net" "net/http" @@ -69,24 +70,10 @@ func log200(r *http.Request) { log.Printf("*** %v :: 200 :: %v %v :: %v\n", uip, r.Method, r.URL, useragent) } -func log400(w http.ResponseWriter, r *http.Request, err string) { - uip := getIPFromCtx(r.Context()) - log.Printf("*** %v :: 400 :: %v %v :: %v\n", uip, r.Method, r.URL, err) - http.Error(w, "400 Bad Request: "+err, http.StatusBadRequest) -} - -func log404(w http.ResponseWriter, r *http.Request, err error) { +func errHTTP(w http.ResponseWriter, r *http.Request, err error, code int) { useragent := r.Header["User-Agent"] - uip := getIPFromCtx(r.Context()) - log.Printf("*** %v :: 404 :: %v %v :: %v :: %v\n", uip, r.Method, r.URL, useragent, err.Error()) - http.Error(w, err.Error(), http.StatusNotFound) -} - -func log500(w http.ResponseWriter, r *http.Request, err error) { - useragent := r.Header["User-Agent"] - uip := getIPFromCtx(r.Context()) - log.Printf("*** %v :: 500 :: %v %v :: %v :: %v\n", uip, r.Method, r.URL, useragent, err.Error()) - http.Error(w, err.Error(), http.StatusInternalServerError) + log.Printf("*** %v :: %v :: %v %v :: %v :: %v\n", uip, code, r.Method, r.URL, useragent, err.Error()) + http.Error(w, fmt.Sprintf("Error %v: %v", code, err.Error()), code) } |