Veo un par de detalles en la forma en la que estás trabajando en la parte del lado servidor.
En primer lugar, tienes una función tipo async
. En la misma usas la palabra await
para realizar la conexión y para ejecutar la consulta.
Si estás usando node-oracledb
(no lo especificas, pero imagino que sí), debes envolver tu código dentro de un bloque try/catch
. Ya que al no hacerlo no estarías capturando un posible error de conexión o un error en la consulta.
Al ser un servicio REST, dichos errores deben ser indicados al cliente usando un estado HTTP 5xx
, ya que los mismos son errores de Servidor. El mensaje que envíes al cliente depende lo que desees exponer de tu API. Te recomiendo mensajes personalizados si tu API es pública, ya que el driver
de Oracle podría mostrar detalles de tu esquema de Base de Datos si envías directamente el mensaje al cliente.
En segundo lugar, como indica @PabloLozano, si la respuesta enviada al cliente incluye un estado 404
, entonces el cliente no recibirá una respuesta válida (HTTP 2xx) por parte de tu API.
Una forma de solventar esto, si deseas mantener el estado 404
para indicar un recurso no encontrado (cosa que no es exactamente lo que sucede), es usar un interceptor, tal como sugiere @PabloLozano.
Otra forma de hacerlo, sería no enviar el estado 404
al cliente cuando la consulta no arroja resultados. En vez de eso envías una respuesta al cliente (por lo general un objeto vacío o un array
vacío), y que sea el cliente el que interprete el resultado obtenido.
Como en tu cliente ya estás probando o comparando objetos, entonces lo ideal sería enviar un objeto vacío.
Tu código del lado servidor podría quedar así:
export async function getGuard(req: Request, res: Response): Promise<any> {
try {
const conn = await connect();
let { documento} = req.params;
const result = await conn.execute(
`
select ideide
from basdat.siide, basdat.siideead
where idecod='${documento}'
and idecod=ideeadcod
and ideeadact='S'
`
);
console.log(result.rows[0][0].length);
// verificamos si NO tenemos resultados de la consulta
if(!result.rows[0][0].length) {
// devolvemos un objeto vacío al cliente
return res.status(200).json({});
}
// si tenemos resultado, enviamos el mismo
return res.status(200).json(result.rows[0][0]);
}
// capturamos la excepción ya sea de conexión o de la consulta
catch(e) {
console.error(e.message);
// enviamos un mensaje personalizado al cliente
return res.status(500).json({error: 'Hubo un problema al realizar la consulta'});
// en caso de que deseemos enviar el mensaje de error devuelto por el driver:
// return res.status(500).json({error: e.message});
// no olvidemos comentar el return anterior si usamos este último.
}
}
De esta forma, cuando recibas una respuesta en tu cliente, tendrás control sobre el resultado de la consulta que recibas, teniendo los siguiente casos:
- Error de servidor mostrado mediante un estado
HTTP 500
.
- Respuesta del Servidor mediante un estado
HTTP 200
, la cual incluye:
- Documento/Registro encontrado en la consulta y devuelto por el Servidor.
Obviamente, si la consulta no devuelve resultados (o devuelve un objeto/array
vacío), significa que el registro no ha sido encontrado en la Base de Datos.
De esta forma separas tus estados tipo 404
que normalmente se refieren a un recurso de red no encontrado, de los errores tipo 500
que son los que suceden cuando el Servidor tiene un fallo interno (no se pudo conectar a la BD o la consulta a la DB devolvió un error).
Si la conexión y consulta a la BD (proceso interno de tu servidor) son satisfactorias, entonces no es necesario devolver un estado diferente del 200
, ya que:
- El recurso (
endpoint
) es accesible.
- La operación de dicho recurso fue satisfactoria.
Espero que esto aclare un poco más tus dudas.