URL Path Parameter get bugged

[constantins2001]

Issue Description

Context#Param(name string) doesn't work in certain constellations.

Checklist

• [ X ] Dependencies installed
• [ X ] No typos
• [ X ] Searched existing issues and docs

Expected behaviour

The following code yields results for handler1 (using c.Param("page")) and for handler3 and handler4 (using c.Param("id"))

	mux.GET("path/:page", handler1)
	mux.POST("path", handler2)
	mux.PATCH("path/:id", handler3)
	mux.DELETE("path/:id", handler4)

Actual behaviour

The following code yields and empty string for handler3 and handler4 (using c.Param("id")).

	mux.GET("path/:page", handler1)
	mux.POST("path", handler2)
	mux.PATCH("path/:id", handler3)
	mux.DELETE("path/:id", handler4)

The following code yields and empty string for handler1 (using c.Param("page")).

	mux.POST("path", handler2)
	mux.PATCH("path/:id", handler3)
	mux.DELETE("path/:id", handler4)
        mux.GET("path/:page", handler1)

Steps to reproduce

func handler3(c echo.Context) error {
	idParam := c.Param("id")
        log.Println(idParam);
}

Working code to debug

See above

Version/commit

Latest (github.com/labstack/echo)

[constantins2001]

After reading through this issue I found that that all parameter names for the same endpoint need to be the same (which was my temporary fix).
It would certainly be advisable to mention this in the docs

[infiz]

I am seeing the same issue and having the workaround as @SaubereSache mentioned. However, it makes the code harder to maintain. It would be good if it can be fixed.

[nhapq]

Revert to version 4.1.5 fix the issue in our case.

echo/router.go

Line 302 in 5c7c87d

func (r *Router) Find(method, path string, c Context) {

[asahasrabuddhe]

Refer #1467 I think your issue should be fixed starting from version 4.1.13

[constantins2001]

Works, thanks for the fix :)

[constantins2001]

Regression, broke again as of v4.1.16 (@asahasrabuddhe @vishr)

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[sm4ll-3gg]

Is it still actual?

[smoyer64]

We ran into this bug again using v4.1.16 on one of our projects. The tests that were added as part of the merge for #1467 only test the GET and POST verbs.

[yoshihiro-shu]

We'are facing this problem at v4.7.2.
how can i solve it?
if someone know how to solve it, could you tell me?

[aldas]

I think this problem was fixed with #2209 in v4.8.0. Related issues #1726 and #2201