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

Already on GitHub? Sign in to your account

Connecting access_token records with existing users #45

wants to merge 3 commits into


None yet
2 participants

I came up with a quick little method to check a facebook users email address agains the existing users table to avoid duplicate user records.

This change would need to be scaled out to all the different oauth and open id providers. Since each provider has a different method to retrieve the users email address my code should probably be refractored.

If you have any insight to this problem let me know. I'd be happy extend this functionality.

Jonathan Spooner and others added some commits Oct 25, 2010

Jonathan Spooner Added https to twitter f32a63c
Jonathan Spooner When the user logs in with facebook I've added a check to see if this…
… user currently has an account in their fb email address. TODO: Add checks for other services like Twitter and possibly find a better location for it
@jspooner jspooner Adding .specification file 21c9169

kimptoc commented May 4, 2011

Sounds good to me :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment