Tengo un problema al intentar hacer una consulta en mongo, el problema es el siguiente:
Intento crear un nuevo usuario, y si este ya existe, devolver un json con un status de error, la inserccion a la bd lo hacer correctamente, el problema esta cuando inserto un usuario ya registrado, que sale el siguiente error en la consola: Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client.
Mi codigo es el siguiente:
function newUser(req, res) {
var params = req.body;
var user = new User();
if (params.name && params.surname && params.nick && params.email && params.password) {
user.name = params.name;
user.surname = params.surname;
user.nick = params.nick;
user.email = params.email;
user.role = 'ROLE_USER';
user.image = null;
User.find({
$or: [
{ email: user.email.toLowerCase() },
{ nick: user.nick.toLowerCase() }
]
}).exec((err, users) => {
if (err) return res.status(500).json({ message: 'Error en la peticion' });
if (users && users.length >= 1) {
return res.status(400).json({ message: 'El usuario ya existe' });
}
});
//Cifra la contraseña y guarda los datos en la bd
bcrypt.hash(params.password, null, null, (err, hash) => {
user.password = hash;
user.save((err, usuarioGuardado) => {
if (err) return res.status(500).send({ message: 'Error al guardar el usuario' });
if (usuarioGuardado) {
res.status(200).send({ user: usuarioGuardado });
} else {
res.status(404).send({ message: 'No se ha registrado el usuario' });
}
});
});
} else {
res.status(200).send({
message: "Rellena todos los campos necesarios!!"
});
}
}
El error solo aparece cuando entra en el if de que el usuario existe, cuando no existe el usuario registrado inserta los datos sin problema. He intentado mil cosas y no logro solucionarlo, gracias de antemano y un saludo.
email
como el camponick
sean de tipounique
, entonces solo basta con realizar un sólo paso:save()
. Si al salvar el documento obtienes un error, basta con determinar si el mismo es sobre la condición de algunos de los campos. Esto evita la doble consulta sobre tu BD. Saludos – Mauricio Contreras el 23 may. 20 a las 14:52hash
delpassword
como un proceso del controlador o de la funciónmiddleware
. Mi sugerencia es delegar esa tarea alSchema
mediante un métodopre
. La idea es encapsular lo máximo posible tu modelo de datos. Saludos – Mauricio Contreras el 23 may. 20 a las 14:58