From 4226a3ab2e1fd8177471e6bb681e1e4ad7acc622 Mon Sep 17 00:00:00 2001
From: Vishal Rana <vr@labstack.com>
Date: Wed, 6 May 2015 10:27:01 -0700
Subject: [PATCH] HTTPError default message

Signed-off-by: Vishal Rana <vr@labstack.com>
---
 echo.go               |  8 ++++++--
 router_test.go        | 35 +++++++++++++++++++++++++++++++----
 website/docs/guide.md |  2 ++
 3 files changed, 39 insertions(+), 6 deletions(-)

diff --git a/echo.go b/echo.go
index ddab9ac0..9d7f2133 100644
--- a/echo.go
+++ b/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)
 	})
diff --git a/router_test.go b/router_test.go
index 804b0969..1736c87e 100644
--- a/router_test.go
+++ b/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")
 	}
diff --git a/website/docs/guide.md b/website/docs/guide.md
index 46cca58a..51c80893 100644
--- a/website/docs/guide.md
+++ b/website/docs/guide.md
@@ -212,4 +212,6 @@ e.Index("index.html")
 
 <!-- ## Error Handling -->
 
+<!-- message not set err.Error() or status text -->
+
 <!-- Deployment -->