[Architecture] Factorisation du site - Githubissues

annuaire-entreprises-data-gouv-fr / site

Vérifiez les informations légales publiques des entreprises, associations et services publics en France

https://annuaire-entreprises.data.gouv.fr

MIT License

49 stars 15 forks source link

[Architecture] Factorisation du site #1101

Closed XavierJp closed 2 months ago

XavierJp commented 3 months ago

Factorisation du dataflow

Nettoyage du erreur flow

Les clients (http)

Lèver des exceptions HTTP -> sous la forme HttpXXX

Model

Convertir les exceptions Http en objet métier APINotResponding

wrappable en un tryer-catcher avec conversion automatique ?

Internals

Route & usefetch

une route ne peut pas lever d'erreur métier
lever une InternalError ou RouteError
c'est une erreur 500 qui ne va pas être logée en front

Proposition de créer objet métier

génère les routes
génère les useFetch
Gestion des droits
- prévenir les appels inutiles dans le useFetch
- interdire les appels non autorisé sur les routes (403)
- withIgnoreBots
- refacto du ProtectedAPIRoute, il faut que par defaut les routes soit protégées et que l'opt out de la protection soit manifeste

Autre

Nettoyage des DataSection #1099
Le model pourrait être étendu avec les sources (et le nom ?) -> pour être utilisé dans le datasection