ForbesLindesay/connect-roles

Doc typo

Closed this issue · 1 comments

In the docs in the dynamic section https://github.com/ForbesLindesay/connect-roles#dynamic

req.params doesn't seem to work. I was testing with restify with is supposed to emulate connect regarding middleware.

I checked your server.js example, and you used this.params, and that worked.

So the two req.params.userid should be this.params.userid

I was banging my head on that for a few minutes...

Good catch, the request object is made available as this inside the authorization strategy. I'll update the readme.