Neues YouTube-Video: Visualisierung im Devices-Adapter

[tueftler17]

In den letzten Tagen habe ich zusammen mit Dirk-Peter ein Problem in einem Adapter behoben. Dabei ist mir aufgefallen, dass ein Aufruf von subscribeForeignStates() mit einem leeren Pattern-String identisch wirkt wie mit einem '*'.

Konkret war es hier dann so, dass unter bestimmten Umständen der User gar nicht die Möglichkeit haben einen State zum abonnieren einzutragen, daher der String leer geblieben ist und dann mit einem leeren String das subscribe aufgerufen wurde.

Das führt bei einer größeren ioBroker-Installation zu einer "Event-Flut", die ein System deutlich verlangsamen kann. Ich hatte dann das Problem, dass teilweise auf Events erst nach 3 Sekunden reagiert wurde. Für technische Dinge mag das ja noch gehen, bei einem Lichtschalter hat es den WAF erheblich reduziert ;-).

Daher mein Vorschlag: Ich empfehle einen Check bei subscribeForeignStates auf einen leeren String und entweder eine Warning- oder gar eine Error-Meldung. Wenn man mit einem * aufruft hat kann das grundsätzlich Sinn haben, aber ich denke: Mit einem leeren String wird in der Regel ein Folgefehler sein.

[paul53]

@tueftler17 [sagte]: Check bei subscribeForeignStates auf einen leeren String und entweder eine Warning- oder gar eine Error-Meldung.

Erstelle Issue auf Github.

[tueftler17]

erledigt

[OliverIO]

@tueftler17

Müsstest du als Adapter Entwickler nicht diesen Check einbauen?
Man lässt Nutzer Eingaben eigentlich nicht ungeprüft auf System Funktionen los.

Man könnte zb ersteinmal prüfen wie viele object ids die Eingabe ergibt und wenn es eine gewisse Grenze überschreitet einen warnhinweis auszugeben ob man das wirklich will

[tueftler17]

@oliverio
Für meinen eigenen Adapter hätte ich ja gar keine Schmerzen damit. Und: Nachdem ich gesehen habe was passiert wenn man einen leeren String in den call gibt weiß ich zu 100%, dass ich das zu verhindern wissen werde. Meine Installation war durch den Fehler soweit kompromittiert, dass der WAF im Keller war und ich ernsthaft überlegt habe auf einen Konkurrenten zu ioBroker zu wechseln weil ich ein 3/4 Jahr das Problem nicht gefunden habe.

Aber: Das Problem trat nicht in meinem Adapter auf und es gibt einen langen Thread dazu wo bei dem betroffenen Adapter schon länger über Performance-Themen gesprochen wurde. Da ist es bisher nicht aufgefallen. Mein Vorschlag ist daher für die gesamte Community gedacht, wenn es einen Eintrag im Log als Error gäbe: Dann wäre das Risiko deutlich geringer, dass dieses an anderen Stellen ebenfalls auffällt.

Ich selber habe bei meinem System noch einen weiteren Fremd-Adapter im Verdacht das zu tun, aber da muss ich ran und ein paar debug-output-Zeilen "reinmogeln" und würde dann auch den Adapter-Entwickler anschreiben. Das sind aber alles Zufallsbefunde und kein strukturiertes Herangehen.

Zwei Zeilen im ioBroker-Core könnten vielleicht zu einer echten Verbesserung führen.

[mcm1957]

@tueftler17 sagte:

In den letzten Tagen habe ich zusammen mit Dirk-Peter ein Problem in einem Adapter behoben. Dabei ist mir aufgefallen, dass ein Aufruf von subscribeForeignStates() mit einem leeren Pattern-String identisch wirkt wie mit einem '*'.

Konkret war es hier dann so, dass unter bestimmten Umständen der User gar nicht die Möglichkeit haben einen State zum abonnieren einzutragen, daher der String leer geblieben ist und dann mit einem leeren String das subscribe aufgerufen wurde.

Das führt bei einer größeren ioBroker-Installation zu einer "Event-Flut", die ein System deutlich verlangsamen kann. Ich hatte dann das Problem, dass teilweise auf Events erst nach 3 Sekunden reagiert wurde. Für technische Dinge mag das ja noch gehen, bei einem Lichtschalter hat es den WAF erheblich reduziert ;-).

Daher mein Vorschlag: Ich empfehle einen Check bei subscribeForeignStates auf einen leeren String und entweder eine Warning- oder gar eine Error-Meldung. Wenn man mit einem * aufruft hat kann das grundsätzlich Sinn haben, aber ich denke: Mit einem leeren String wird in der Regel ein Folgefehler sein.

Bitte issue im js-controller anlegen.

[tueftler17]

Hab ich heute Vormittag schon gemacht.
https://github.com/ioBroker/ioBroker.js-controller/issues/3325