Función Async o Sync bcrypt para usar en node.js para generar hashes?

Question

Actualmente estoy intentando hacer un módulo de autenticación para mi proyecto en node.js?

ya he visto algunos ejemplos del uso de bcrypt para generar valores hash, es decir

https://github.com/bnoguchi/mongoose-auth/blob/master/lib/modules/password/plugin.js https://github.com/Turbo87/locomotive-passport-boilerplate/blob/master/app/models/account.js

Sin embargo, por alguna razón que están utilizando la función bcrypt.hashSync(). Desde bcrypt es bueno porque es mucho tiempo, ¿no sería más prudente utilizar la función asíncrona lugar con el fin de no bloquear el código, es decir:

User.virtual('password') 
.get(function() { 
    return this.hash; 
}) 
.set(function (password) { 
    bcrypt.hash('password', 10, function(err, hash) { 
     this.hash = hash; 
    }); 
}); 

¿Me puede explicar qué camino es mejor y por qué? ¡Gracias!

Answer 1

Sí, querría usar la versión asíncrona si es posible para que no esté atando el procesamiento de su nodo durante el hash de la contraseña. En ambos casos de código fuente a los que hace referencia, el código está utilizando la versión síncrona porque el método en el que se utiliza es sincrónico, por lo que el autor no tuvo más remedio que utilizar la versión síncrona.

Answer 2

No puede realizar una llamada asíncrona dentro de un método síncrono. Intente crear un método diferente para usar al establecer la contraseña.

Acabo de enviar una solicitud de extracción para que el proyecto de alguien haga exactamente esto. Compruébelo aquí: https://github.com/nickpoorman/CrowdNotes/commit/e268c80a9cacddbc0215bf0e2b7aa31c0a4c785f