MyElectricalData / myelectricaldata_import

134 stars 35 forks source link

[BUG] - config.example.yaml format url exemple web service incorrect #569

Open typhoe opened 3 months ago

typhoe commented 3 months ago

Description du bug L'exemple fourni pour l'url du websocket homeassistant dans le fichier de configuration prête a confusion

https://github.com/MyElectricalData/myelectricaldata_import/blob/652ca166920d532f043ffa14a99326d53e65a750/config.example.yaml#L13

En effet, l'exemple semble indiquer de mettre ws:// mais dans ce cas, la connexion échoue (le ws:// est rajouté dans le code de l'appli, non ?)

2024-08-03 17:18:25.952 -     INFO : [MON_PDL] IMPORT DES DONNÉES VERS L'ONGLET ENERGY DE HOME ASSISTANT (WEBSOCKET)
2024-08-03 17:18:25.952 -     INFO : ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ ◦ ❖ ◦ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
2024-08-03 17:18:25.953 -     INFO : Connexion au WebSocket Home Assistant ws://ws://homeassistant:8123/api/websocket
2024-08-03 17:18:29.708 -    ERROR : 
2024-08-03 17:18:29.708 -    ERROR :     Impossible de se connecter au WebSocket Home Assistant.
2024-08-03 17:18:29.708 -    ERROR : 
2024-08-03 17:18:29.708 -    ERROR :     Vous pouvez récupérer un exemple ici :
2024-08-03 17:18:29.708 -    ERROR : https://github.com/MyElectricalData/myelectricaldata_import/blob/main/config.example.yaml
2024-08-03 17:18:29.708 -    ERROR : 
2024-08-03 17:18:29.708 - CRITICAL : La configuration Home Assistant WebSocket est erronée
Configuration (config.yaml) Fonctionnel (sans `ws://`) ```yaml home_assistant_ws: enable: true ssl: false token: '' url: homeassistant:8123 ``` Non Fonctionnel (avec `ws://`) ```yaml home_assistant_ws: enable: true ssl: false token: '' url: ws://homeassistant:8123 ```

Type d'installation

Version : 1.0.0rc14

cvalentin-dkt commented 2 months ago

Je viens de fix en local et ce sera update dans la prochaine release :)