Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajout de logs lors de la géneration de documents #613

Closed
jusabatier opened this issue Nov 24, 2021 · 9 comments
Closed

Ajout de logs lors de la géneration de documents #613

jusabatier opened this issue Nov 24, 2021 · 9 comments
Assignees
Milestone

Comments

@jusabatier
Copy link
Collaborator

Comme évoqué précédemment, il semble intéressant d'avoir des logs concernant les documents générés par l'application :

  • Bordereau parcellaires
  • Relevé de propriété
  • Demandes d'infos
  • Exports

Ceci permettrait d'une part de garder une trace des infos qui sortent de l'appli, et d'autre part de récupérer des métriques d'utilisation de l'outil.

@landryb
Copy link
Member

landryb commented Nov 29, 2021

pi je viens de faire une metrique arbitraire en greppant mes logs, depuis septembre 2020 environ 7000 BP et 12000 RP.

@MaelREBOUX
Copy link
Member

Je rêve de retrouver ce genre de stats

@MaelREBOUX
Copy link
Member

tu partages ton grep @landryb ?

@landryb
Copy link
Member

landryb commented Nov 30, 2021

nbp=0
nrp=0
for f in /srv/log/tomcat/cadastrapp*gz ; do

date=$(basename $f .log.gz)
# utilisateurs authentifiés par jour
zgrep ROLE $f  | cut -d ' ' -f7 | sort -u > /tmp/cadalog/users-$date
zgrep -c -e /createBordereauParcellaire.*response $f > /tmp/cadalog/bp-$date
nbp=$(( $nbp + $(cat /tmp/cadalog/bp-$date) ))
zgrep -c -e /createRelevePropriete.*response $f > /tmp/cadalog/rp-$date
nrp=$(( $nrp + $(cat /tmp/cadalog/rp-$date) ))
done
echo bp:$nbp
echo rp:$nrp

@pierrejego pierrejego self-assigned this Feb 21, 2022
@pierrejego
Copy link
Member

@landryb @MaelREBOUX @catmorales vous en pensez quoi.

Pour moi c'est un gros plus, je reteste et je vérifie si ça s'installe bien (activé/desactivé)

Côte RGPD vous êtes bien tous d'accord.

Je propose d'en reparler ensemble et de l'intégrer dans la prochaine release.

@pierrejego pierrejego added this to the v 2.1 milestone Feb 21, 2022
@MaelREBOUX
Copy link
Member

La PR est #612

@landryb
Copy link
Member

landryb commented Feb 21, 2022

perso j'ai jamais été fan des logs en DB

@jusabatier
Copy link
Collaborator Author

Ça n'a rien d'obligatoire, ça permet juste à ceux que ça intéresse de les mettre en place.

Le principe c'est juste de passer à log4j2 (plus récent que l'actuel logback), et ajouter des fonctions de log de niveau INFO.
De plus, le namespace de log utilisé est org.georchestra.cadastrapp.loggers.documents ce qui permet de le désactiver en le passant en WARN.

Enfin j'ai fourni en commentaire un exemple de config pour log dans BDD, afin de guider ceux qui voudraient le mettre en place.

@MaelREBOUX
Copy link
Member

Mergé donc on ferme

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants