Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Commit

Permalink
Merge pull request #46 from memolog/fix-twitter-strategy
Browse files Browse the repository at this point in the history
Use twitter.id_str instead of twitter.id in twitter strategy, great catch @memolog!
  • Loading branch information
amoshaviv committed Aug 27, 2013
2 parents ebd0f7e + e566939 commit df6ab28
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion config/passport.js
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ module.exports = function(passport, config) {
},
function(token, tokenSecret, profile, done) {
User.findOne({
'twitter.id': profile.id
'twitter.id_str': profile.id
}, function(err, user) {
if (err) {
return done(err);
Expand Down

0 comments on commit df6ab28

Please sign in to comment.