認証モジュール 'Passport'は、ログインを行うためにFindOrCreateメソッドが必要です。私は、次のスキーマと私のユーザーを保存するためにマングースを使用しています:非同期の処理方法。パスポートとマングースのfindOrCreateメソッド
var UserSchema = new Schema({
firstname: String,
lastname: String,
email: String,
accounts: []
});
アカウントの配列は{provider: "facebook", uid: "someFacebookId"}
のように、Facebookのアカウントを表すオブジェクトを保持しています。
マイ認証戦略は次のようになります。
// 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);
});
}
});
});
}
));
問題:を私のデータベース(User.find(...)
)コールバック関数が私のデータベースが応答するのを待つことなく、すぐに実行された後に照会します。これにより、定義されていないolduser
オブジェクトが生成されます。だから私はこのユーザーがログインしようとするたびに、同じユーザーの別名を自分のデータベースに入れています。
この非同期コールバックを適切に処理する方法を教えてください。
私はこれが質問に直接関係しているわけではありませんが、それは少し危険ですか?指定された値のaccounts.uidを持つユーザーと、「facebook」のaccounts.providerを持つユーザーを探します。しかし、それらを同じアカウントリスト要素にするのはなぜですか?つまり、別のユーザーが別のプロバイダと一致するUIDを持っていたらどうでしょうか? – StevenC
私は両方の値の組み合わせを探していると仮定していますが、これは一意でなければなりません。 – Sven
これは危険であると仮定します。なぜなら、ユーザーがFacebookアカウントを持っていて、* ANY *アカウントにそのuidがある場合、アカウントの配列が一致するからです。誰かがOpenAuthサーバを持っているなら、彼は望みのユーザIDを返すことで任意のユーザとしてログインできます。 – tangxinfa