search

ban-archive / api-gestion-poc

POC pour une API de gestion BAN
15 stars 11 forks source link

Exposer l'"insee" courant en header #242

Open yohanboniface opened 7 years ago

yohanboniface commented 7 years ago

Comme pour le user et client, on veut exposer cette info dans les logs.

odorie commented 7 years ago

il me semble que celà n'a pas été fait. Cette exposition était compliquée à mettre en place car risquait de ralentir l'API ?

yohanboniface commented 7 years ago

Cette exposition était compliquée à mettre en place car risquait de ralentir l'API ?

C'est surtout que ça demande un peu redesign (pour factoriser le code), et qu'il y a des cas qui ne sont pas évidents à trancher: si par exemple je demande un PostCode qui est sur deux communes, dois-je loguer les deux communes? Si je demande une liste de group ou housenumbers, et que dans cette liste il y a des rues de dix communes différentes, dois-je loguer les dix communes?