allowing logged-in user to bind a passport without username or email

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

allowing logged-in user to bind a passport without username or email #159

Closed maslennikov closed 1 year ago

maslennikov commented 7 years ago

The reasoning behind this change is that we need unique user credentials provided in auth profile only for the case when there is no corresponding passport stored in DB and the current user is not authenticated.

Live example: I am authenticated and I want to connect my facebook profile with my user, and facebook for some reason doesn't give me the email/username. Basically, I don't care for the email right now because the current user has a verified email already attached.

TemaSM commented 7 years ago

@maslennikov Thanks for your code :wink: @tjwebb May you please merge it?

tjwebb commented 7 years ago

This branch has conflicts that must be resolved