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

ajoute le timestamp du log globalement #53

Closed
wants to merge 1 commit into from

Conversation

Bigsmooth68
Copy link
Contributor

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
Copy link
Owner

bokub commented Jul 8, 2024

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
Copy link
Contributor Author

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
Copy link
Owner

bokub commented Jul 8, 2024

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:

  • Le plus lisible possible: éviter les détails superflus, être compréhensibles par une personne lambda
  • Suffisamment fournis pour que je puisse traiter toutes les demandes d'aides. Le jour et l'heure sont importants pour comprendre le contexte, mais les secondes n'ont aucun intérêt particulier

@Bigsmooth68 Bigsmooth68 closed this Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants