Bluebird Promisify mehrere Argumente

Question

Ich bin neu zu Versprechen, und ich weiß nicht, wie dieses Problem zu beheben: Ich mache ein Auth-System, und mein erster Aufruf ist, E-Mails in der Datenbank zu überprüfen. Wenn ein Benutzer existiert, dann überprüfe das Passwort mit einem verschlüsselten Passwort ... Ich benutze diese lib für bcrypt: https://npmjs.org/package/bcrypt was nicht kompatibel ist, also benutze ich "promisify" für die folgende Signatur: compare (password, crypted_password, Rückrufen).

Also das ist mein Code:

var compare = Promise.promisify(bcrypt.compare); 

User.findByEmail(email) 
    .then(compare()) <--- here is the problem 

Das ist meine findByEmail Methode:

User.prototype.findByEmail = function(email) { 
var resolver = Promise.pending(); 

knex('users') 
    .where({'email': email}) 
    .select() 
    .then(function(user) { 
     if (_.isEmpty(user)) { resolver.reject('User not found'); } 
     resolver.fulfill(user); 
    }); 


return resolver.promise; 

}

Wie mehr Werte des "Vergleich" Verfahren in diesem Fall zuweisen? Verpasse ich den Punkt der Versprechen?

Answer 1

.then(compare()) <--- here is the problem 

Die then method hat eine Funktion erwarten, die ein weiteres Versprechen zurück [oder einen einfachen Wert], so dass Sie compare es ohne Aufruf zu passieren brauchen würde. Wenn Sie die Argumente angeben müssen, verwenden Sie eine Wrapper-Funktion Ausdruck:

User.findByEmail(email) 
    .then(function(user) { 
     return compare(/* magic */); 
    }).… 

Answer 2

Ich tat genau das, was Bergi sagte und arbeitet für mich:

this.findByEmail(email) 
.then(function(user) { 
    return compare(password, user.password); 
})