Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

koa-passport@next is broken! #139

Open
sergeyampo opened this issue Apr 3, 2020 · 1 comment
Open

koa-passport@next is broken! #139

sergeyampo opened this issue Apr 3, 2020 · 1 comment

Comments

@sergeyampo
Copy link

Spent a lot of time with debugging and looking for a problem with @next version:
I used a lot of strategies(local, JWT...) with passed callback in passport.authenticate function and can't understand why (error, user) => { } parameters in the callback works so unpredictable.
For example, I've passed such arguments to the done callback in the strategy: done(null, user); after that passport.authenticate callback will be called with (user, undefined).
So I have to pass such arguments to make it work properly: done(null, null, user) or done(null, error). Sometimes, some of the parameters set to false. There's no such errors in main version of koa-passport!

@rkusa
Copy link
Owner

rkusa commented Apr 3, 2020

Hi, the next version is currently not used for an upcoming version and thus outdated. Please use latest (latest is on 4.1.3 while next is 2.2.2)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants