fix: handling error on route backend#241
Conversation
- Add decorator to handle Integrity Error or other error on process CRUD - Add decorator on delete route for organisms and users Reviewed-by: andriac
andriacap
force-pushed
the
fix/handling-error-route-backend
branch
from
d321c48 to
290d671
Compare
There was a problem hiding this comment.
L'idée est bonne ! Toutefois, je pense que la déclaration du décorateurs sur les routes concernées n'est pas idéal. Je pense qu'il est plus simple de s'appuyer sur la mécanique Flask des error_handlers https://flask.palletsprojects.com/en/stable/errorhandling/.
L'idée serait d'appeller une fonction à chaque erreur 500. Dans cette fonction, si l'erreur est une erreur SQLA, on affiche le message d'erreur (dans un flash).
app/t_roles/route.py
Outdated
| 6, | ||
| ) | ||
| @fnauth.check_auth(6) | ||
| @handle_db_errors("user.users", "l'utilisateur", redirect_on_error=True) |
There was a problem hiding this comment.
Lourd si on doit déclarer ça plusieurs fois ...
There was a problem hiding this comment.
J'ai proposé une solution centralisée en terme de gestion d'erreurs
There was a problem hiding this comment.
Ola ola ! Un petit UP au cas où c'était passé à la trap :)
andriacap
force-pushed
the
fix/handling-error-route-backend
branch
from
ab37e68 to
fc80d3e
Compare
- Add handler function according to current function existing in errors.py Reviewed-by: andriac
andriacap
force-pushed
the
fix/handling-error-route-backend
branch
from
fc80d3e to
b36db95
Compare
|
Hello , Je me permet de relancer le sujet ici concernant des messages d'erreur explicite au lieu de "Internal Server Error" . NOtamment quand on veut supprimer un utilisateur qui est associé à des observations . Pas mal d'utilisateur veulent "nettoyer" des anciens utilisateurs mais ne comprennent pas pourquoi ils peuvent pas supprimer ces utilisateurs (car ça leur retourne un message brut Internal Server Error) . Merci d'avance ! |
2 participants
Before when error happened
After with handle error
Reviewed-by: andriacap