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

improving security #35

Open
andri-anski opened this issue Oct 18, 2021 · 6 comments
Open

improving security #35

andri-anski opened this issue Oct 18, 2021 · 6 comments
Labels
enhancement New feature or request

Comments

@andri-anski
Copy link

Leider (oder zum Glück) ist die Bluetooth-Kommunikation bei den Thermostaten nicht verschlüsselt.
Worstcase wäre, wenn man nach dem Urlaub feststellt, dass die Thermostate ununterbrochen aufgedreht waren, weil jemand sich einen Spaß erlaubt hat.
Es wäre sicher nützlich frühzeitig feststellen zu können, wenn die Sollwerte des Thermostats durch fremde Einwirkung verändert wurden.
Prinzipiell würde dafür auch ein zusätzlicher State als Indikator reichen.

@Schnup89
Copy link
Owner

Gute Idee, ich würde dafür einen state erstellen "force_temperature" der standardmäßig auf false steht.
So kann man pro Thermostat entscheiden ob die Temperatur beim zyklischen auslesen neu gesetzt werden soll falls diese vom temp wert abweicht

@andri-anski
Copy link
Author

andri-anski commented Oct 19, 2021

Das wäre sehr nützlich. Vor allem dann, wenn die Bedienung vollständig über Iobroker erfolgen soll. In diesem Fall werden alle Veränderungen, ob am Thermostat oder über Fremdeingriff, der Sollwerte sofort rückgängig gemacht. Das wäre OK. interessant wäre möglicherweise noch folgendes:

  • Zu berücksichtigen ist auch die Boost Funktion. Wenn diese An ist, verändert sich nicht Temperatur-Soll. Demnach, sollte bei jedem Auslesen auch Boost überprüft werden.

  • Ein zusätzliche Maßnahme in Verbindung mit force_temperatur wäre, die Bedienung am Thermostat zu blockieren (Lock). Dann kann sich Gewissheit einstellen, dass die Soll-Werte nicht am Thermostat verändert wurden.

  • Ein zusätlicher State als Indikator für Fremdeinwirkung wäre möglicherweise doch dann wünschenswert, um zu erfahren, wenn jemand am System rumspielt.

Sehr gute Arbeit, gefällt mir!

@Schnup89 Schnup89 added the enhancement New feature or request label Oct 20, 2021
@Schnup89
Copy link
Owner

Schnup89 commented Nov 1, 2021

Hatte es heute morgen versucht zu implementieren, das Auslesen der States per getStateAsync ist ein kraus...
Ich werde das wohl nicht hinbekommen, evtl. erbarmt sich jemand der sich mit der Materie besser auskennt als ich :)

@herrniffler
Copy link
Collaborator

Ich wollte mal ein paar lokale Änderungen hier rein mergen, da schau ich das gleich mit an. Kann aber ein paar Tage/Wochen daueren, ich muss schauen wie ich Zeit finde

@herrniffler
Copy link
Collaborator

@Schnup89 wie würdes du force_temperatur setzten? über die config seite oder als state den man dann manuel anpassen kann?

@Schnup89
Copy link
Owner

Ich denke über einen state ist es einfacher, über die config Seite muss man die liste HTML und und anpassen. Danke für deine hilfe!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants