Incorrect TSR with mixed static and wildcard paths

[ibraheemdev]

The backtracking logic in #2897 is causing incorrect TSR values:

func TestTreeBacktrackingTsr(t *testing.T) {
	tree := &node{}
	tree.addRoute("/api/:page/:name", fakeHandler("1"))
	tree.addRoute("/api/hello/:name/bar/", fakeHandler("1"))
	res := tree.getValue("/api/hello/x/bar", getParams(), getSkippedNodes(), false)
	if res.tsr != true {
		t.Fatalf("expected tsr")
	}
}

--- FAIL: TestTreeBacktrackingTsr (0.00s)
    tree_test.go:855: expected tsr

I believe the TSR would have to be checked before backtracking.

[ibraheemdev]

Also, I don't know why this unrelated change was made in #2767:

-	value.tsr = (path == "/") ||
-			(len(prefix) == len(path)+1 && prefix[len(path)] == '/' &&
-				path == prefix[:len(prefix)-1] && n.handlers != nil)
+		value.tsr = path == "/" ||
+			(len(prefix) == len(path)+1 && n.handlers != nil)

It is causing simple tests to fail:

func TestTreeTsr(t *testing.T) {
	tree := &node{}
	tree.addRoute("/foo", fakeHandler("1"))
	tree.addRoute("/bar/", fakeHandler("1"))
	res := tree.getValue("/baz", getParams(), getSkippedNodes(), false)
	if res.tsr != false {
		t.Fatalf("unexpected tsr")
	}
}

--- FAIL: TestTreeTsr (0.00s)
    tree_test.go:855: unexpected tsr

[ibraheemdev]

I have this fixed locally, I'll submit a PR.

[Bisstocuz]

I have this fixed locally, I'll submit a PR.

Great works, waiting for it.
#2847 fixed the second issue so you don't need to submit it again I think.

[zhuxi0511]

I have this fixed locally, I'll submit a PR.

Thanks for your work, waiting for it!