Skip to content

Fix timezone issue in daemon logs#54

Open
TiTidom-RC wants to merge 1 commit intojeedom:betafrom
TiTidom-RC:beta
Open

Fix timezone issue in daemon logs#54
TiTidom-RC wants to merge 1 commit intojeedom:betafrom
TiTidom-RC:beta

Conversation

@TiTidom-RC
Copy link

Description

Les logs du démon mqtt2d s'affichaient en heure GMT/UTC au lieu d'utiliser l'heure locale du système Jeedom.

Problème :
Les timestamps dans les logs utilisaient new Date().toISOString() qui retourne systématiquement l'heure en UTC, indépendamment du fuseau horaire configuré dans Jeedom.

Solution :
Remplacement de toISOString() par une fonction de formatage qui utilise les méthodes de Date() respectant automatiquement le fuseau horaire local du système.

Fichier modifié : resources/mqtt2d/jeedom/jeedom.js

Suggested changelog entry

Passage des logs du démon en heure locale.

Related issues/external references

Fixes #

Types of changes

  • Bug fix (non-breaking change which fixes)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
    • This change is only breaking for integrators, not for external standards or end-users.
  • Documentation improvement

PR checklist

@zoic21
Copy link
Contributor

zoic21 commented Mar 13, 2026

Bonjour,
C'est voulu tous les demon log toujours en UTC (c'est plus simple pour faire du debug dessus en utc).

@TiTidom-RC
Copy link
Author

Bonsoir,

Compliqué de faire de la corrélation de logs entre le plugin et le démon si l'un est en UTC et l'autre dans la timezone du jeedom.

et la plupart des utilisateurs ne voient que le fait que "ce n'est pas à la bonne heure" :)

@zoic21
Copy link
Contributor

zoic21 commented Mar 14, 2026

Je sais mais ça a toujours été comme ça, l’heure utc est fiable quelques soit la configuration de la timezone. Vue qu’on est sur des démons donc du bas niveau on veut avoir un truc fiable et non dépendant d’une configuration utilisateur

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