Tengo este bloque try-catch que se supone que detecta cualquier error encontrado al conectarse a una base de datos Mongo y devolver el estado de "No autorizado". Sin embargo, Express falla en el backend, aunque estoy manejando los errores si apago el motor MongoDB para realizar pruebas. ¿No deberían manejarse correctamente los errores? (No estoy usando el código de error a partir de ahora para probar).
app.post('/api/login', (req, res) => { id = req.body.account var query = {_id: id} try { db.collection("Users").findOne(query, (err, result) => { if (result) { nonce = noncef() var upvalues = { $set: {nonce: nonce} } db.collection("Users").updateOne(query, upvalues, (err, result) => { if (result) { res.send(`${nonce}`) } }) } }) } catch(err) { res.status(401).send('Unauthorized') } })Use Promises para .findOne y .updateOne en lugar de devoluciones de llamada, y luego puede await los resultados (y capturarlos) o encadenar un .catch en la cadena Promise.
app.post('/api/login', async (req, res) => { const query = { _id: req.body.account } try { const user = await db.collection("Users").findOne(query); if (!user) return; const nonce = noncef(); const upvalues = { $set: { nonce } }; const updated = await db.collection("Users").updateOne(query, upvalues); if (updated) res.send(nonce); } catch (err) { res.status(401).send('Unauthorized') } })En general, en JavaScript moderno, siempre debe preferir las promesas a las devoluciones de llamada cuando sea posible: las promesas hacen que el encadenamiento y la captura de código asincrónico sean mucho más convenientes.