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. Saludoshash
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