Merge pull request #629 from kiyonlin/reset-ctx-matched

🔔 fix ctx matched bug
gofiber · Jul 17, 2020 · 531e50e · 531e50e · Fenny · Jul 17, 2020
2 parents 6fd1dd9 + c89bfb3
commit 531e50e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/app_test.go b/app_test.go
@@ -40,7 +40,12 @@ func Test_App_MethodNotAllowed(t *testing.T) {
 
 	app.Options("/", func(c *Ctx) {})
 
-	resp, err := app.Test(httptest.NewRequest("GET", "/", nil))
+	resp, err := app.Test(httptest.NewRequest("POST", "/", nil))
+	utils.AssertEqual(t, nil, err)
+	utils.AssertEqual(t, 200, resp.StatusCode)
+	utils.AssertEqual(t, "", resp.Header.Get(HeaderAllow))
+
+	resp, err = app.Test(httptest.NewRequest("GET", "/", nil))
 	utils.AssertEqual(t, nil, err)
 	utils.AssertEqual(t, 405, resp.StatusCode)
 	utils.AssertEqual(t, "POST, OPTIONS", resp.Header.Get(HeaderAllow))

diff --git a/ctx.go b/ctx.go
@@ -87,6 +87,8 @@ func (app *App) AcquireCtx(fctx *fasthttp.RequestCtx) *Ctx {
 	// Reset route and handler index
 	ctx.indexRoute = -1
 	ctx.indexHandler = 0
+	// Reset matched flag
+	ctx.matched = false
 	// Set paths
 	ctx.path = getString(fctx.URI().PathOriginal())
 	ctx.pathOriginal = ctx.path