Tengo un controlador de errores de middleware que funciona muy bien, pero next(err) y return y return next(err) parecen no detener la ejecución cuando se trata de promesas. ¿Cuál es la forma correcta de detener la ejecución de mi código cuando se encuentra un error?
Para referencia: err es una clase de error estándar en este caso.
No creo que necesites el código en userProvider.fetchFriends para ayudarte con esto, pero si no es así, házmelo saber.
const uid = req.query.steamUserId; //Use the userProvider to get steam friend data const friendData = await userProvider.fetchFriends(uid) .catch(err => { return next(err); //Does not stop execution. Why? just next(err) doesn't either. }); //Putting this after every catch works but seems very stupid. How do I avoid this? if(res.writableEnded) return; ...Other code that runs but causes errors }Tienes dos problemas aquí.
Primero: next() continúa explícitamente con el siguiente middleware o punto final.
Para terminar de atender la solicitud, envía una respuesta.
const middleware = (req, res, next) => { if (something(req)) { next(); } else { res.status(500).json({ ok: false, msg: "some error message" }); } }Segundo: debe observar cuidadosamente su lógica asíncrona.
no puedes:
Solo puede enviar una respuesta a una solicitud.
O:
next o res.something en el bloque catch y simplemente registre el error internamente onext o res.something fuera del manejo de la promesa y muévalo a un controlador then (es posible que desee cambiar a usar async / await para que su lógica sea más fácil de seguir)El problema era que estaba mezclando async/await con .then/.catch. Necesario para usar try/catch.
ty @jonsharpe
export const getSteamFriends = async (req, res, next) => { try{ const uid = req.query.steamUserId; //Use the userProvider to get steam friend data const friendData = await userProvider.fetchFriends(uid); //more code in the middle } catch(e) { return next(e); } };