bokub / ha-linky

📊 Home Assistant add-on for Linky smart meters
MIT License
192 stars 9 forks source link

ajoute le timestamp du log globalement #53

Closed Bigsmooth68 closed 2 months ago

Bigsmooth68 commented 2 months ago

Bonjour, J'ai noté que les timestamps des logs ne sont pas sur tous les messages. J'ai donc retiré les timestamps d'index.ts pour les ajouter à log.ts. Je ne suis pas sûr que la PR peut être accepté tel quelle, mais ca donne une idée de comment faire. J'ai également ajouté les secondes.

bokub commented 2 months ago

En effet, les timestamps ne sont pas sur tous les logs mais uniquement sur le premier d'une série de logs

Par exemple: tu démarres ton add-on le soir à 19h, il va y avoir un premier log qui affiche l'heure, puis une dizaine de logs liés au même évènement. Plus rien ne se passe jusqu'au lendemain matin à 6h30, une synchronisation se lance, un premier log affiche l'heure courante, puis une dizaine de logs liés à cette synchronisation

Je ne suis pas certain de comprendre l'intérêt d'afficher l'heure systématiquement, encore moins les secondes, mais je veux bien connaitre les motivations de ta PR

Bigsmooth68 commented 2 months ago

Mon idée est d'avoir des logs cohérents. La plupart des applications logs un format standard: timestamp, log level, message Déformation professionelle, sans doute :)

bokub commented 2 months ago

Dans un contexte professionnel, les logs sont récoltés bruts et détaillés, mais ont généralement vocation à être traités et parsés pour une exploitation ultérieure (debug, statistiques, recherche, etc..)

Dans ce cas précis, les logs font office d'interface avec l'utilisateur final, le but est donc qu'ils soient à la fois: