Echo ignores path param declarations for the same URL if a previous one exists

[ivange94]

Hello,

I noticed a behavior with echo that wasn't documented and I'm wondering if this is a bug or design.

Say for instance I have a route like this

e.GET("/users/:id", func (c echo.Context) error {
    id := c.Param("id")
    fmt.Printf("id: %v\n", 
})

A user made a PR with a route like this

e.PUT("/users/:userId", func(c echo.Context) error {
      id := c.Param("userId")
      fmt.Printf("id: %v\n", id)
})

The latter did not work and it took me a few minutes to realize that once you have an endpoint registered with a path param with a particular name, subsequent endpoints will use that and ignore any new names.

Ideally all parameters of same purpose will have the same name but I'm wondering if this behavior is intentional. If it's intentional maybe an error should be thrown when users mistakenly change the name?

[aldas]

This is a known "feature" of router. See #1744 , #1726 This "feature" has been reworked in v5 so every route can have their own path parameter names.