jfrchicanog / ewp-reference-connector

MIT License
0 stars 0 forks source link

Tratamiento de errores #37

Closed jfrchicanog closed 3 years ago

jfrchicanog commented 3 years ago

https://github.com/jfrchicanog/ewp-reference-connector/blob/c9b96961117150fdf0b645fb9726eb643750175e/ewp-reference-connector/src/main/java/eu/erasmuswithoutpaper/iia/approval/boundary/IiaApprovalResource.java#L116

Aquí hay varias cosas que indicar.

  1. ¿Lo único que puede pasar es que no esté en Algoria? El endpoint podría estar caído (aunque esto no da un error 400), el token podría estar mal porque se ha actualizado en Algoria.
  2. No se debe dar un error al socio porque la API interna hacia Algoria no funciona, esto es un detalle interno del que el socio no tiene culpa. Por tanto, si la conexión con Algoria falla, hay que guardar la aprobación (en cualquier caso hay que hacerlo) y responder afirmativamente al socio. Luego se intenta la conexión con Algoria más tarde.
  3. No debería intentar comunicarse con Algoria en la resolución de la petición. Se debe programar como un trabajo para realizar y repetir si es necesario. Puede usarse la Bath API de Java para esto.
Dalia-git commented 3 years ago

Programacion cambiada. De todas formas en lo que habia antes solo veias el mensaje claro del acuerdo no encontrado pq para el resto Algoria me daba una respuesta detallada . Es que Manuel me habia comentado que aun no habia tenido en cuenta que el acuerdo no se encontrara en Algoria:

Dalia-git commented 3 years ago

El resto de lo errores tratados por Algoria son estos segun me comunica Manuel: