search

yukulehe / gazpar2mqtt

Python script to fetch GRDF's website data and publish data to a mqtt broker.
GNU General Public License v3.0
21 stars 9 forks source link

Keep secret #24

Closed aladin2000 closed 2 years ago

aladin2000 commented 2 years ago

bonjour à toi je voulais savoir si on pouvait avoir les infos de connexion de cet addon gardées secrétes dans HA comme par exemple

GRDF_USERNAME: !secret grdf.username [11:01:12] FATAL: gazpar.username does not follow the structure KEY=text, it will be ignored and removed from the config

yukulehe commented 2 years ago

@alexbelgium Salut Alex, penses-tu pouvoir faire quelque chose pour aladin ?

alexbelgium commented 2 years ago

Bonjour, quel est l'inconvénient de l'avoir dans le log ? Car personne n'y a accès à part toi ? C'est juste pour comprendre le soucis car techniquement ce sera faisable

yukulehe commented 2 years ago

Je n'ai peut être pas compris ta réponse Alex et je ne connais pas bien non plus la partie add-on de HA. Mais je suppose que c'est pour avoir un docker_compose qui ne stocke pas le password ? Du coup ça se passe de quel côté pour gérer les paramètres !secret ? Add-on ou appli ?

aladin2000 commented 2 years ago

Bjr Pour être conforme aux logiques présentes depuis le début dans ha Tous mes mdp sont codes dans un seul fichier secrets.yaml

Plus difficile à lire pour autrui et eviter d'avoir des mdp partout Si c'est pas difficile le repérage du tag secret qui donne l'ouverture du fichier centralisé. ...

1 déc. 2021 12:02:01 Alexandre @.***>:

Bonjour, quel est l'inconvénient de l'avoir dans le log ? Car personne n'y a accès à part toi ? C'est juste pour comprendre le soucis car techniquement ce sera faisable

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub[https://github.com/yukulehe/gazpar2mqtt/issues/24#issuecomment-983526331], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AD7K4I5SMFJKUKK6SFAFLQ3UOX6CPANCNFSM5JECZD4Q]. Triage notifications on the go with GitHub Mobile for iOS[https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675] or Android[https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub]. [data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEQAAABECAYAAAA4E5OyAAAAAXNSR0IArs4c6QAAAARzQklUCAgICHwIZIgAAAAoSURBVHic7cEBAQAAAIIg/69uSEABAAAAAAAAAAAAAAAAAAAAAACPBkiEAAHzeIrHAAAAAElFTkSuQmCC###24x24:true###][Image de pistage][https://github.com/notifications/beacon/AD7K4I23BNSW7D3ZYIAYM7DUOX6CPA5CNFSM5JECZD42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHKPWXOY.gif]

alexbelgium commented 2 years ago

Je vais pusher une version "0.5_dev_dontupdate", pourrais tu tester? C'était au final beaucoup plus compliqué que ce que je pensais à implémenter mais cela pourrait fonctionner. Ma seule crainte est que cela interfère avec les autres variables. Merci d'avance pour le test

EDIT : le test ne fonctionne pas je dois investiguer plus en profondeur

aladin2000 commented 2 years ago

je reste à ta disposition... Cela concernerait également le adon enedis2mqtt ... ou le token, le PDL sont aussi en clair....

alexbelgium commented 2 years ago

J'ai pushé une version test si tu as l'occasion de tester ce serait top

aladin2000 commented 2 years ago

Je le fais DQP

1 déc. 2021 23:21:19 Alexandre @.***>:

J'ai pushé une version test si tu as l'occasion de tester ce serait top

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub[https://github.com/yukulehe/gazpar2mqtt/issues/24#issuecomment-984105972], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AD7K4IZZ646HY4HSUQFVRQDUO2NV5ANCNFSM5JECZD4Q]. Triage notifications on the go with GitHub Mobile for iOS[https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675] or Android[https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub]. [data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEQAAABECAYAAAA4E5OyAAAAAXNSR0IArs4c6QAAAARzQklUCAgICHwIZIgAAAAoSURBVHic7cEBAQAAAIIg/69uSEABAAAAAAAAAAAAAAAAAAAAAACPBkiEAAHzeIrHAAAAAElFTkSuQmCC###24x24:true###][Image de pistage][https://github.com/notifications/beacon/AD7K4I5PG5R7SNPY55FD6K3UO2NV5A5CNFSM5JECZD42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHKUEH5A.gif]

aladin2000 commented 2 years ago

Merci bcp Cela fonctionne avec. Ta dernière version

1 déc. 2021 23:55:59 Aladin @.***>:

Je le fais DQP

1 déc. 2021 23:21:19 Alexandre @.***>:

J'ai pushé une version test si tu as l'occasion de tester ce serait top

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub[https://github.com/yukulehe/gazpar2mqtt/issues/24#issuecomment-984105972], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AD7K4IZZ646HY4HSUQFVRQDUO2NV5ANCNFSM5JECZD4Q]. Triage notifications on the go with GitHub Mobile for iOS[https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675] or Android[https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub]. [data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEQAAABECAYAAAA4E5OyAAAAAXNSR0IArs4c6QAAAARzQklUCAgICHwIZIgAAAAoSURBVHic7cEBAQAAAIIg/69uSEABAAAAAAAAAAAAAAAAAAAAAACPBkiEAAHzeIrHAAAAAElFTkSuQmCC###24x24:true###][Image de pistage][https://github.com/notifications/beacon/AD7K4I5PG5R7SNPY55FD6K3UO2NV5A5CNFSM5JECZD42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHKUEH5A.gif]

alexbelgium commented 2 years ago

Sorry, ça faisait crasher d'autres (https://github.com/alexbelgium/hassio-addons/issues/99) j'ai du enlever l'option secrets. Je vais refaire des tests...

aladin2000 commented 2 years ago

Secret C'est ok si je mets pas à jour .... ?

2 déc. 2021 11:33:13 Alexandre @.***>:

Sorry, ça faisait crasher d'autres (alexbelgium/hassio-addons#99[https://github.com/alexbelgium/hassio-addons/issues/99]) j'ai du enlever l'option secrets. Je vais refaire des tests...

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub[https://github.com/yukulehe/gazpar2mqtt/issues/24#issuecomment-984499527], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AD7K4I3RI6GRFKIWMSSJ563UO5DOPANCNFSM5JECZD4Q]. Triage notifications on the go with GitHub Mobile for iOS[https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675] or Android[https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub]. [data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAEQAAABECAYAAAA4E5OyAAAAAXNSR0IArs4c6QAAAARzQklUCAgICHwIZIgAAAAoSURBVHic7cEBAQAAAIIg/69uSEABAAAAAAAAAAAAAAAAAAAAAACPBkiEAAHzeIrHAAAAAElFTkSuQmCC###24x24:true###][Image de pistage][https://github.com/notifications/beacon/AD7K4I2C2QGYCIOPWLTI4JDUO5DOPA5CNFSM5JECZD42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHKXEKRY.gif]

alexbelgium commented 2 years ago

Bonne idée. Je te fais signe quand une nouvelle version avec secret est en ligne

alexbelgium commented 2 years ago

Bonjour, la version 0.5.1-4 devrait maintenant fonctionner pour tout le monde et inclue le mode secret

yukulehe commented 2 years ago

https://github.com/alexbelgium/hassio-addons/issues/99