Los errores que deberías mostrar, en un primer lugar, deben ser de validación. Los errores internos no deben ser expuestos al cliente porque a éste no le interesan los detalles, además, usuarios mal intencionados pueden aprovechar ésta información para abrir brechas de seguridad.
Los errores de validación soy muy importantes en un sistema, ya que así mostramos al usuario las reglas que debe seguir al intentar persistir datos. Mongoose tiene una jerarquía de errors, desde MongoseError
hasta ValidationError
. Cada hijo dentro de la jerarquía hereda la propiedad errors
, el cual es un array de mensajes de error.
Manejando errores a nivel de base de datos
Un problema que tienes es que estás devolviendo un error dentro de un callback, lo cual no logrará el efecto deseado. En su lugar, debes manejar el error dentro del callback y mostrar una respuesta la usuario o algo.
mongoose
.connect(...)
.then(() => {
connected = true;
})
.catch(err => {
connected = false;
});
Si usas Express o Koa puedes hacer un middleware para que verifique la conexión a la base de datos y responder de acuerdo a ella:
app.use('*', (req, res, next) => {
if (!connected) {
return res.status(500).send("Couldn't stablish connection. Please, contact the admin");
} else {
return next();
}
});
De ésta manera, si hacemos una petición a cualquier URL del sistema y ha ocurrido un error de autenticación con la base de datos, obtendremos dicho mensaje seguido de un error 500 (interval server error).
Manejando errores de validación de modelos
A nivel de validación de esquemas/modelos, lo único que debemos hacer es enviar los mensajes de errors que Mongoose ha elaborado por nosotros de acuerdo a las restricciones en los esquemas. Por ejemplo tenemos el siguiente esquema:
const UserSchema = new Schema({
username: {
type: Schema.Types.String,
required: true,
unique: true
}
...
});
Si insertamos un username
duplicado, obtendremos un ValidationError
el cual podemos manejar:
User
.save({ ... })
.then((user) => {
res.jsonp(user);
})
.catch(err => {
res.jsonp({
success: false,
errors: err.errors
});
});