Param returning empty string

[tekknolagi]

I modified the example project a bit, adding this:

func nextUserId() string {
    var next = nextid;
    nextid += 1;
    return strconv.Itoa(next)
}

func createUser(c *echo.Context) {
    u := new(user)
    u.ID = nextUserId()
    u.Name = c.Param("name")

    if c.Bind(u) {
        users[u.ID] = *u
        c.JSON(http.StatusCreated, u)
    }
}

Except u.Name is for some reason always the empty string. What am I doing wrong? I've looked at the context.go source and it looks like it should work.

[vishr]

From the example, createUser handler doesn't accept param.

[tekknolagi]

I've modified the handlers like so:

e.Get("/users", getUsers)
e.Get("/users/:id", getUser)

e.Post("/users", createUser)
e.Post("/users/:name", createUser)

but now get a Method Not Allowed. If I comment out the /users/:id route, it works.

[vishr]

Can you paste the full code and how you invoke the endpoint?

[tekknolagi]

https://gist.github.com/e168fc8fe9be985901fd

[vishr]

e.Get("/users/:id", getUser)
e.Post("/users/:name", createUser)

With current implementation, above is registered as conflicting routes, so if you change :name to :id, it should work. Let me know your thoughts.

Conflicting routes are captured in #3

[tekknolagi]

Is it possible to do this without a pretty URL? Just /users?name=joel?

Since

e.Get("/users/:id", getUser)
e.Post("/users/:name", createUser)

should not be conflicting routes.

[vishr]

Based on this example https://golang.org/pkg/net/url/#example_URL

c.Request.URL.Query().Get("user")

[tekknolagi]

@vishr I'm not sure what you are trying to say.

I meant that in most other frameworks, those two routes are distinct.

I unfortunately still get an empty string.

[tekknolagi]

https://gist.github.com/471fb336acb72f02d726

[vishr]

Just realized, it won't work. I will work on fixing the implementation to accommodate this scenario.

[tekknolagi]

Including the duplication of routes?

[vishr]

Duplicate routes like below will be covered in #3

e.Get("users/:id")
e.Get("users/:id")
e.Get("users/:name")

[tekknolagi]

Now I just cannot connect to port 8080 (with the same code).

[vishr]

I tried your original code https://gist.github.com/e168fc8fe9be985901fd for all the registered routes. Output below:

➜  tmp  curl -X GET localhost:8080/users
{"1":{"id":"1","name":"Wreck-It Ralph"}}
➜  tmp  curl -X GET localhost:8080/users/1
{"id":"1","name":"Wreck-It Ralph"}
➜  tmp  curl -X POST localhost:8080/users
{"id":"2","name":""}
➜  tmp  curl -X POST localhost:8080/users/Joe
{"id":"3","name":"Joe"}

Did you update Echo to latest?