Disable Router.SaveMatchedRoute by default

clevergo · Feb 5, 2020 · a057e20 · a057e20
1 parent ad95961
commit a057e20
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 1 deletion.
diff --git a/route_test.go b/route_test.go
@@ -285,6 +285,7 @@ func ExampleRouteGroup() {
 
 func ExampleGetRoute() {
 	router := NewRouter()
+	router.SaveMatchedRoute = true
 	router.Get("/posts/:page", func(_ http.ResponseWriter, r *http.Request) {
 		ps := GetParams(r)
 		page, _ := ps.Int("page")

diff --git a/router.go b/router.go
@@ -158,7 +158,6 @@ func NewRouter() *Router {
 		RedirectFixedPath:      true,
 		HandleMethodNotAllowed: true,
 		HandleOPTIONS:          true,
-		SaveMatchedRoute:       true,
 	}
 }
 

diff --git a/router_test.go b/router_test.go
@@ -736,6 +736,7 @@ func TestRouterMatchedRoutePath(t *testing.T) {
 	}
 
 	router := NewRouter()
+	router.SaveMatchedRoute = true
 	router.HandleFunc(http.MethodGet, route1, handle1)
 	router.HandleFunc(http.MethodGet, route2, handle2)
 	router.HandleFunc(http.MethodGet, route3, handle3)