context doesn't capture "Group" params about echo

[vishr]

@tve Let me think over it, I never thought this way ;)

[tve]

Cool, thanks for fixing!
You still have a problem with a trailing slash:

$ curl http://localhost:4444/acct/123/hello/
2015/05/04 21:44:20 http: panic serving [::1]:38441: runtime error: index out of range
goroutine 6 [running]:
net/http.func·011()
        /big/usr-local/go/src/net/http/server.go:1130 +0xbb
github.com/labstack/echo.(*router).Find(0xc20800a750, 0xc20801e4e0, 0x3, 0xc20801e4e4, 0x10, 0xc208040550, 0x0, 0x0)
        /home/src/github.com/labstack/echo/router.go:256 +0x407
github.com/labstack/echo.(*Echo).ServeHTTP(0xc208066000, 0x7f5e901a70d0, 0xc2080443c0, 0xc208032820)
        /home/src/github.com/labstack/echo/echo.go:296 +0xd0

[tve]

Can you ack this issue or do I need to fork & fix myself? Thanks!

[vishr]

I tried to fix it but it opened a can of worms. I am gonna give it a shot over the weekend, just hang on.

[vishr]

c58ec74

[vishr]

18dcc4b

Potentially a similar issue, I'm seeing group middleware with a wildcard group.Get match return the wrong url segment in the param value.

// instead of getting "labstack" from c.Param("thing"), I get "rules"
GET /v1/labstack/rules

func main() {
  e = echo.New()
  g := e.Group("/v1/:thing")
  g.Use(ThingMiddleware)
  g.Get("/*", Handle)
  e.Run(":1234")
}

func ThingMiddleware() echo.HandlerFunc {
  return func(c *echo.Context) error {
    log.Println(c.Param("thing"))
    return nil
  }
}

func Handle(c *echo.Context) error {
  return c.String(200, c.Param("thing"))
}

[vishr]

@laicosly Looks like a bug when used with wildcard, it's the same even without using a group e.g. e.Get("/v1/:thing/*, Handle). Can you create a separate issue?