trailsjs / sails-auth

Passport-based User Authentication system for sails.js applications. Designed to work well with the sails-permissions module.
https://www.npmjs.org/package/sails-auth
MIT License
265 stars 141 forks source link

Get error while "An existing user is trying to log in using an alread… #129

Closed litinskii closed 8 years ago

litinskii commented 8 years ago

…y connected passport." #128

fenichelar commented 8 years ago

Can we get this approved?

litinskii commented 8 years ago

Approver of what ? Show you error when i try to do this? For example. Passport model isn't populate to user model so you can't get passport.user.id

fenichelar commented 8 years ago

I meant "accept" the pull request, I mistyped "approved". Thanks for making the fix.

litinskii commented 8 years ago

I haven't "write access" to do this. Or i didn't understand about what "accept" the pull request you talking about.

fenichelar commented 8 years ago

I know. I wasn't referring to you :)

litinskii commented 8 years ago

Sorry for the misunderstanding, this is my mistake.

fenichelar commented 8 years ago

No worries. I couldn't figure out how to fix the problem, you got it working. Works flawlessly!

jials commented 8 years ago

This fix makes it work again! Thanks!

basert commented 8 years ago

Please merge this fix, thanks :)

basert commented 8 years ago

Wow, thanks for the fast merge, can you create a new npm version aswell? When I specify the github repo by hand, the dist folder does not exist and starting the app fails with: Error: Cannot find module '...node_modules/sails-auth'] code: 'MODULE_NOT_FOUND'

tjwebb commented 8 years ago

You likely will need to run npm install inside the repo first to run the build

kiel-cristian commented 8 years ago

@tjwebb Waiting also for new npm version aswell, please!

Great work with this package, it saves me a lot of working time :)

ahmadao commented 7 years ago

I still get this error after logging out, then logging in again.

ahmadao commented 7 years ago

The list inside the package.json points to a tarball that doesn't contain the latest version. How can we fix this?

ahmadao commented 7 years ago

The npm version is older than the git version. How can we get the latest version?

anabelle commented 7 years ago

+1