search

etalab / transport-site

Rendre disponible, valoriser et améliorer les données transports
https://transport.data.gouv.fr
198 stars 30 forks source link

Ajout Plug.Telemetry #4191

Closed AntoineAugusti closed 2 months ago

AntoineAugusti commented 2 months ago

Fixes #4190

Ce plug a été ajouté dans Phoenix 1.5.0 et nous n'avons pas fait cette ajout. Ceci a posé des difficultés avec notre APM AppSignal et pourrait avoir d'autres impacts.

AntoineAugusti commented 2 months ago

@thbar La décision d'ajouter ce plug te parait bien ? Voir l'issue.

AntoineAugusti commented 2 months ago

Pour ce type de changement, je serais tenté de faire un petit push sur le staging et voir si ça réagit bien.

Bonne idée ! Faudrait pas qu'on ait des doublons d'événements dans AppSignal. Je vais faire ça.

AntoineAugusti commented 2 months ago

J'ai bien pu vérifier qu'il n'y avait pas de doublon d'événement HTTP en staging et que ça remontait bien dans AppSignal

thbar commented 2 months ago

Impec, top !