diff --git a/error.go b/error.go index e1d6dfae..e2848e57 100644 --- a/error.go +++ b/error.go @@ -204,11 +204,7 @@ type errorInfo struct { } // map of http handlers for each error string. -var ErrorMaps map[string]*errorInfo - -func init() { - ErrorMaps = make(map[string]*errorInfo) -} +var ErrorMaps map[string]*errorInfo = make(map[string]*errorInfo, 10) // show 401 unauthorized error. func unauthorized(rw http.ResponseWriter, r *http.Request) { @@ -392,22 +388,21 @@ func ErrorController(c ControllerInterface) *App { } // show error string as simple text message. -// if error string is empty, show 500 error as default. -func exception(errcode string, ctx *context.Context) { - code, err := strconv.Atoi(errcode) - if err != nil { - code = 503 - } - if h, ok := ErrorMaps[errcode]; ok { - executeError(h, ctx, code) - return - } else if h, ok := ErrorMaps["503"]; ok { - executeError(h, ctx, code) - return - } else { - ctx.ResponseWriter.WriteHeader(code) - ctx.WriteString(errcode) +// if error string is empty, show 503 or 500 error as default. +func exception(errCode string, ctx *context.Context) { + for ec, _ := range []string{errCode, "503", "500"} { + code, _ := strconv.Atoi(errCode) + if code == 0 { + code = 503 + } + if h, ok := ErrorMaps[ec]; ok { + executeError(h, ctx, code) + return + } } + //if 50x error has been removed from errorMap + ctx.ResponseWriter.WriteHeader(503) + ctx.WriteString(errCode) } func executeError(err *errorInfo, ctx *context.Context, code int) {