Wie mit Async umzugehen. findOrCreate-Methode für Pass und Mungo

Question

Das Authentifizierungsmodul 'Passport' erfordert eine FindOrCreate-Methode, um eine Anmeldung durchzuführen. Ich bin mit Mungo, um meine Benutzer mit dem folgenden Schema zu speichern:

var UserSchema = new Schema({ 
    firstname: String, 
    lastname: String, 
    email: String, 
    accounts: [] 
}); 

Die Konten Array-Objekte enthält, die Facebook-Konten darstellen, wie {provider: "facebook", uid: "someFacebookId"}.

Meine Authentifizierungsstrategie sieht wie folgt aus:

// Authentication Strategy 
passport.use(new FacebookStrategy({ 
    clientID: CONFIG.fb.appId, 
    clientSecret: CONFIG.fb.appSecret, 
    callbackURL: CONFIG.fb.callbackURL 
    }, 
    function(accessToken, refreshToken, profile, done) { 
    // asynchronous verification, for effect... 
    process.nextTick(function() { 

     User.find({ 'accounts.uid': profile.id, 'accounts.provider': 'facebook' }, function(err, olduser) { 

      if(olduser._id) { 
      console.log('User: ' + olduser.firstname + ' ' + olduser.lastname + ' found and logged in!'); 
      done(null, olduser); 
      } else { 
      var newuser = new User(); 
      var account = {provider: "facebook", uid: profile.id}; 
      newuser.accounts.push(account); 
      newuser.firstname = profile.name.givenName; 
      newuser.lastname = profile.name.familyName; 
      newuser.email = "TBD..."; 

      newuser.save(function(err) { 
       if(err) { throw err; } 
       console.log('New user: ' + newuser.firstname + ' ' + newuser.lastname + ' created and logged in!'); 
       done(null, newuser); 
      }); 
      } 
     }); 
    }); 
    } 
)); 

Problem: Nach meiner Datenbank abfragt (User.find(...)) die Callback-Funktion sofort ausgeführt wird, ohne zu warten, für meine Datenbank zu beantworten. Dies führt zu einem undefinierten Objekt olduser. Daher erhalte ich jedes Mal, wenn dieser Benutzer versucht, sich anzumelden, eine Kopie desselben Benutzers in meine Datenbank.

Wie gehe ich richtig mit diesem asynchronen Rückruf um?

Answer 1

User.find gibt ein Array von Dokumenten zurück, die Ihren Bedingungen entsprechen. In Ihrem Fall möchten Sie stattdessen User.findOne verwenden und dann if (olduser)... überprüfen, um festzustellen, ob ein übereinstimmendes Dokument gefunden wurde.

Answer 2

process.nextTick(function() { 
     var query = User.findOne({ 'fbId': profile.id }); 
     query.exec(function (err, oldUser) { 
     console.log(oldUser); 
     if(oldUser) { 
      console.log('User: ' + oldUser.name + ' found and logged in!'); 
      done(null, oldUser); 
     } else { 
      var newUser = new User(); 
      newUser.fbId = profile.id; 
      newUser.name = profile.displayName; 
      newUser.email = profile.emails[0].value; 

      newUser.save(function(err) { 
      if(err) {throw err;} 
      console.log('New user: ' + newUser.name + ' created and logged in!'); 
      done(null, newUser); 
      }); 
     } 
     }); 
    }); 

Answer 3

Hass nitpick, aber die anderen erwähnten Methoden hier brechen, wenn zwei Benutzer gleichzeitig Zeit-- anmeldet und versuchen, bevor Sie in Produktion gehen, werden Sie einen Blick in Transaktionen haben wollen: http://www.mongodb.org/display/DOCS/two-phase+commit

Answer 4

Es gibt eine App dafür: mongoose-findorcreate