SIU-Toba / framework

Framework para desarrollo rápido de aplicaciones web
http://toba.siu.edu.ar
21 stars 24 forks source link

HTTP Response codes #17

Open lgraziani2712 opened 6 years ago

lgraziani2712 commented 6 years ago

Buenas!

Estuve viendo que siempre se responde con el código 200 aún cuando hay errores.

Por ejemplo en la clase nucleo/componentes/interface/toba_ci, en la línea 873, se lanza un error toba_error_seguridad con un mensaje que dice que el método no existe.

El código HTTP que el cliente recibe es 200, cuando debería ser 501. Por otro lado, ese mensaje nunca llega al cliente.

Justo este error es de una consulta AJAX, y no se devuelve ningún objeto de error.

¿Tienen en mente desarrollar algo relacionado a esto?

Saludos!

enfoqueNativo commented 6 years ago

Hola Luciano,

con respecto al status, se envia siempre 200 incluso para los errores porque son errores manejados por el framework, lo único que te deja sin esa respuesta seria un fatal, es una decision de diseño que se tomo hace mucho tiempo.

El tema del mensaje es un punto a analizar, el mecanismo que se usa es el mismo que para informar cualquier mensaje en un pedido de pagina, en estos casos se vuelve un problema ya que termina rompiendo el parseo. Habria que modificarlo para que se pueda previsualizar el mensaje correctamente.

Voy a mirarlo para ver que implicancias tiene del lado cliente (hay varios puntos desde donde se disparan pedidos xhr) y cuanto hay que modificar, calculo que al menos deberia ir un JSON con el msg de error.

Saludos