gaspersic/freeCodeCamp
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

start making improvements to auth and get email from Twitter users

Browse Source
This commit is contained in:
Michael Q Larson
2014-12-03 23:28:10 -08:00
parent 01e86c74bc
commit 711904d1c1
1 changed files with 18 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
config/passport.js
View File

@ -41,10 +41,10 @@ passport.deserializeUser(function(id, done) {
passport.use(new TwitterStrategy(secrets.twitter, function(req, accessToken, tokenSecret, profile, done) { passport.use(new TwitterStrategy(secrets.twitter, function(req, accessToken, tokenSecret, profile, done) {
if (req.user) { if (req.user) {
User.findOne({ twitter: profile.id }, function(err, existingUser) { User.findOne({ twitter: profile.id }, function(err, existingUser) {
//if (existingUser) { if (existingUser) {
// req.flash('errors', { msg: 'There is already a Twitter account that belongs to you. Sign in with that account or delete it, then link it with your current account.' }); req.flash('errors', { msg: 'There is already a Twitter account that belongs to you. Sign in with that account or delete it, then link it with your current account.' });
// done(err); done(err);
//} else { } else {
User.findById(req.user.id, function(err, user) { User.findById(req.user.id, function(err, user) {
user.twitter = profile.id; user.twitter = profile.id;
user.tokens.push({ kind: 'twitter', accessToken: accessToken, tokenSecret: tokenSecret }); user.tokens.push({ kind: 'twitter', accessToken: accessToken, tokenSecret: tokenSecret });
@ -57,25 +57,30 @@ passport.use(new TwitterStrategy(secrets.twitter, function(req, accessToken, tok
done(err, user); done(err, user);
}); });
}); });
//} }
}); });
} else { } else {
User.findOne({ twitter: profile.id }, function(err, existingUser) { User.findOne({ twitter: profile.id }, function(err, existingUser) {
if (existingUser) return done(null, existingUser); //if (existingUser) return done(null, existingUser);
var user = new User();
// Twitter will not provide an email address. Period. // Twitter will not provide an email address. Period.
// But a person’s twitter username is guaranteed to be unique // But a person’s twitter username is guaranteed to be unique
// so we can "fake" a twitter email address as follows: // so we can "fake" a twitter email address as follows:
//user.email = profile.username + "@twitter.com"; //user.email = profile.username + "@twitter.com";
var user = existingUser || new User;
user.twitter = profile.id; user.twitter = profile.id;
user.email = user.email || '';
user.tokens.push({ kind: 'twitter', accessToken: accessToken, tokenSecret: tokenSecret }); user.tokens.push({ kind: 'twitter', accessToken: accessToken, tokenSecret: tokenSecret });
user.profile.name = profile.displayName; user.profile.name = user.profile.name || profile.displayName;
user.profile.location = profile._json.location; user.profile.location = user.profile.location || profile._json.location;
user.profile.picture = profile._json.profile_image_url_https; user.profile.picture = user.profile.picture || profile._json.profile_image_url_https;
user.save(function(err) { user.save(function(err) {
done(err, user); done(err, user);
}); });
if (!user.email) {
res.redirect('/account');
req.flash('errors', { msg: 'OK, you are signed in. Please add your email address to your profile.' });
}
}); });
} }
})); }));
@ -109,7 +114,7 @@ passport.use(new LinkedInStrategy(secrets.linkedin, function(req, accessToken, r
var user = existingEmailUser || new User; var user = existingEmailUser || new User;
user.linkedin = profile.id; user.linkedin = profile.id;
user.tokens.push({ kind: 'linkedin', accessToken: accessToken }); user.tokens.push({ kind: 'linkedin', accessToken: accessToken });
user.email = profile._json.emailAddress; user.email = user.email || profile._json.emailAddress;
user.profile.name = user.profile.name || profile.displayName; user.profile.name = user.profile.name || profile.displayName;
user.profile.location = user.profile.location || profile._json.location.name; user.profile.location = user.profile.location || profile._json.location.name;
user.profile.picture = user.profile.picture || profile._json.pictureUrl; user.profile.picture = user.profile.picture || profile._json.pictureUrl;
@ -209,7 +214,7 @@ passport.use(new GitHubStrategy(secrets.github, function(req, accessToken, refre
if (existingUser) return done(null, existingUser); if (existingUser) return done(null, existingUser);
User.findOne({ email: profile._json.email }, function(err, existingEmailUser) { User.findOne({ email: profile._json.email }, function(err, existingEmailUser) {
var user = existingEmailUser || new User; var user = existingEmailUser || new User;
user.email = profile._json.email; user.email = user.email || profile._json.email;
user.github = profile.id; user.github = profile.id;
user.tokens.push({ kind: 'github', accessToken: accessToken }); user.tokens.push({ kind: 'github', accessToken: accessToken });
user.profile.name = user.profile.name || profile.displayName; user.profile.name = user.profile.name || profile.displayName;
@ -251,7 +256,7 @@ passport.use(new GoogleStrategy(secrets.google, function(req, accessToken, refre
if (existingUser) return done(null, existingUser); if (existingUser) return done(null, existingUser);
User.findOne({ email: profile._json.email }, function(err, existingEmailUser) { User.findOne({ email: profile._json.email }, function(err, existingEmailUser) {
var user = existingEmailUser || new User; var user = existingEmailUser || new User;
user.email = profile._json.email; user.email = user.email || profile._json.email;
user.google = profile.id; user.google = profile.id;
user.tokens.push({ kind: 'google', accessToken: accessToken }); user.tokens.push({ kind: 'google', accessToken: accessToken });
user.profile.name = user.profile.name || profile.displayName; user.profile.name = user.profile.name || profile.displayName;