HTTPError default message

Signed-off-by: Vishal Rana <[email protected]>

Author: vishr

echo.go

@@ -133,8 +133,12 @@ func New() (e *Echo) {
 		if he.Code == 0 {
 			he.Code = http.StatusInternalServerError
 		}
-		if he.Message == "" && he.Error != nil {
-			he.Message = he.Error.Error()
+		if he.Message == "" {
+			if he.Error != nil {
+				he.Message = he.Error.Error()
+			} else {
+				he.Message = http.StatusText(he.Code)
+			}
 		}
 		http.Error(c.Response, he.Message, he.Code)
 	})

router_test.go

@@ -633,9 +633,34 @@ func TestRouterExperiment(t *testing.T) {
 	r.Add(GET, "/users/", func(*Context) *HTTPError {
 		return nil
 	}, nil)
-	r.Add(GET, "/users/new/*", func(*Context) *HTTPError {
-		return nil
-	}, nil)
+	// r.Add(GET, "/use", func(*Context) error {
+	// 	return nil
+	// }, nil)
+	// r.Add(GET, "/users/*", func(*Context) error {
+	// 	return nil
+	// }, nil)
+	// r.Add(GET, "/users/", func(*Context) error {
+	// 	return nil
+	// }, nil)
+	// r.Add(GET, "/users/new/*", func(*Context) error {
+	// 	return nil
+	// }, nil)
+	// r.Add(GET, "/users/new", func(*Context) error {
+	// 	return nil
+	// }, nil)
+	// r.Add(GET, "/users/:uid", func(*Context) error {
+	// 	return nil
+	// }, nil)
+	// r.Add(GET, "/users/new/:id", func(*Context) error {
+	// 	return nil
+	// }, nil)
+	// r.Add(GET, "/users/wen", func(*Context) error {
+	// 	return nil
+	// }, nil)
+	// r.Add(GET, "/users/:uid/files/:fid", func(*Context) error {
+	// 	return nil
+	// }, nil)
+
 	r.Add(GET, "/users/new", func(*Context) *HTTPError {
 		return nil
 	}, nil)
@@ -652,7 +677,9 @@ func TestRouterExperiment(t *testing.T) {
 		return nil
 	}, nil)
 
-	h, _ := r.Find(GET, "/users/", context)
+	r.trees[GET].printTree("", true)
+
+	h, _ := r.Find(GET, "/users/new", context)
 	if h == nil {
 		t.Fatal("handler not found")
 	}

website/docs/guide.md

@@ -212,4 +212,6 @@ e.Index("index.html")
 
 <!-- ## Error Handling -->
 
+<!-- message not set err.Error() or status text -->
+
 <!-- Deployment -->