Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

Exposer l'"insee" courant en header #242

Open
yohanboniface opened this issue Nov 8, 2016 · 2 comments
Open

Exposer l'"insee" courant en header #242

yohanboniface opened this issue Nov 8, 2016 · 2 comments

Comments

@yohanboniface
Copy link
Contributor

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

@odorie
Copy link
Contributor

odorie commented May 18, 2017

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

@odorie odorie added this to the prod2 milestone May 18, 2017
@yohanboniface
Copy link
Contributor Author

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?

@gschittek gschittek modified the milestones: prod2, prod future Feb 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants