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

Migration vers influxdata/influxdb-client-php v3 et adaptation du code dans cmd.class.php #3039

Open
kwizer15 opened this issue Feb 22, 2025 · 0 comments

Comments

@kwizer15
Copy link
Contributor

kwizer15 commented Feb 22, 2025

Votre demande de fonctionnalité est-elle liée à un problème ? Veuillez le décrire.
Oui, cette demande est liée à un problème de dépendance dans le projet. Actuellement, le code dans cmd.class.php repose sur la bibliothèque obsolète "influxdb/influxdb-php": "^1.15", alors que la dépendance initialement spécifiée dans composer.json était "influxdata/influxdb-client-php": "^3". Cette dernière est une version moderne et maintenue, mais elle est incompatible avec le code actuel, ce qui a nécessité une correction temporaire (via une PR) pour revenir à la version 1.15. Cela crée une dette technique, car nous utilisons une bibliothèque ancienne au lieu de tirer parti des améliorations de la version récente.

Décrivez la solution que vous souhaitez
Je propose de migrer le code de la classe cmd.class.php pour qu’il soit compatible avec "influxdata/influxdb-client-php": "^3". Cela inclut :

  1. Mettre à jour la dépendance dans composer.json pour rétablir "influxdata/influxdb-client-php": "^3".
  2. Adapter les appels aux méthodes de la bibliothèque dans cmd.class.php pour utiliser la nouvelle API de la version 3 (par exemple, gestion des clients, écriture des données, etc.).
  3. Tester les fonctionnalités concernées pour s’assurer qu’il n’y a pas de régression.

Décrivez les alternatives que vous avez envisagées

  1. Rester sur "influxdb/influxdb-php": "^1.15" indéfiniment : Cette option évite un travail immédiat, mais elle maintient une dépendance obsolète, ce qui pourrait poser des problèmes de sécurité ou de compatibilité à l’avenir.
  2. Utiliser une version intermédiaire (par exemple, série 2.x si elle existe) : Cela pourrait réduire l’effort de migration, mais je n’ai pas trouvé de version stable entre les deux qui soit largement adoptée, et cela ne résout pas complètement la dette technique.

Contexte supplémentaire
La bibliothèque "influxdb/influxdb-php": "^1.15" n’est plus activement maintenue (dernière mise à jour significative en 2018), tandis que "influxdata/influxdb-client-php" est la bibliothèque officielle recommandée par InfluxData pour interagir avec InfluxDB, avec une prise en charge des versions récentes du serveur (comme InfluxDB 2.x). Cette migration permettrait à Jeedom de rester à jour avec les standards actuels et de bénéficier de meilleures performances et fonctionnalités.

Voir la PR de "revert" vers "influxdb/influxdb-php": "^1.15" #3038

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

No branches or pull requests

1 participant