Je suis en train de découvrir jeedom connect et à l’installation du plugin j’ai remarqué un « souci » pour ceux utilisant le plugin zigbee.
Le port 8089 est utilisé pour la première antenne, avec une 2e antenne le port 8090 est utilisé, ce qui est en conflit avec celui du jeedom connect. il peut avoir max 3 antennes. Le mieux serait de prendre le port 8092 s’il est dispo.
Alors oui il suffit de changer le port de jeedom-connect manuellement et basta. Mais peut-être ajouter une vérification à l’activation du module pour ceux qui sont moins technique ?
Le fait que des gens ne savent pas que le port 8090 est déjà utilisé ailleurs ou lire des logs.
C’était au passage si le rapprochement n’avait jamais été évoqué. Après on prend ou on prend pas
Peut-être que ces personnes tomberont sur ce sujet et ce sera suffisant
ca c’est un autre sujet
si les gens ne savent pas lire les logs quand il y a un probleme … c’est en effet un soucis !
La proposition que tu fais n’est pas si évidente que ca en faite …
tous les plugins pourraient potentiellement utiliser un port, donc on s’amuserait à partir de 8090 à faire +1 le temps d’en trouver un de dispo ?
de plus peut être que le port est dispo aujourd’hui, mais que tu l’utilises déjà sur un plugin que tu as temporairement désactivé et qui posera problème demain.
sans parler qu’il faut peut etre que tu ouvres des ports sur ton routeur, s’il a été défini automatiquement, il faut pouvoir y avoir accès pour savoir lequel à finalement été choisi pour que tu le mettes sur ton routeur. ==> c’est donc aussi rapide que tu en définisses un toi même
quid de si tu installes un nouveau plugin qui utilise le port choisi automatiquement et qu’il n’est pas paramétrable sur ton nouveau plugin. ?!
enfin bon ca me semble un peu compliqué … pour juste renseigner un champ dans la configuration du plugin ==> 1ère page sur laquelle tu dois aller pour activer ton plugin !
a la limite je préfère faire quelques lignes supplémentaires dans la doc pour montrer une log qui dirait que le port est déjà utilisé, et bien re-préciser où renseigner/personnaliser ce port.
si les gens ne savent pas lire les logs quand il y a un probleme … c’est en effet un soucis !
On est d’accord ^^ mais il y a tout type de gens avec ou pas un gout pour l’analyse
Je me suis posé les mêmes questions pour l’implémentation (mais suis juste ops et pas dev xD)
Pour moi le plus simple serait un test que c’est bien le daemon du plugin qui est sur le port 8090. Afficher un message warning quand l’utilisateur est à l’instant T dans la configuration du plugin signalant que le port est déjà utilisé par un autre processus. Et après il se débrouille
PS : au passage je suis en train de faire les widgets et quelques tests, une pépite ce plugin et l’app! Il y a le temps du setup évidemment mais après je pense qu’une fois que ca roule je serais tranquille! Et on peut faire du custom c’est vraiment top!
Si le port est occupé, le démon est en erreur. Un utilisateur un minimum curieux ira forcément voir les log et devrait voir ce qu’il se passe je suppose…
Je ne suis pas très favorable à le changer par défaut.
Si demain un autre plugin utilise le nouveau port on va s’amuser à le changer de nouveau !
Un petit message d’avertissement pour moi ça me paraît bien dans le sens où tout le monde ne sait pas forcément se retrouver dans les logs. Par contre on est capable d’aller le changer dans la config si on sait que c’est ça le problème…
Je pense aussi a un de mes collègues qui s’est mis à la domotique et qui n’est pas du tout debug, je suppose qu’il est pas le seul xD et c’est moi le support
PS : au passage, je me demande si vous êtes ouverts pour enrichir la doc du github ? Car j’ai un peu de temps. Je pensais notamment aux notifications avec l’app (plusieurs canal pour les notifications différentes c’est vraiment un must), et d’autres trucs qui me viendront en tête.
Après peut-être aussi montrer plusieurs exemples d’imbrication de plugins, idée de remontée via sous-titre, etc mais ça à voir si tuto forum ou doc…
En principe la partie notif est déjà faite via le lien qu a partagé Jared !
Mais tu peux enrichir si tu le souhaites. (L ideal serait de faire 1 Pull-Request par sujet, de facon a ce qus ce soit plus simple pour nous ensuite de les merger)
Ya plusieurs dernieres nouveautés qu on n a pas encore mises/expliquées dans la doc.