search

naimo84 / node-red-contrib-pi-hole-remote

Node RED module for controlling a remote pi-hole
MIT License
10 stars 2 forks source link

Feature Request #2 #7

Closed nicedevil007 closed 4 years ago

nicedevil007 commented 4 years ago

Denkste es ist möglich, dass deine Node automatisch die Anzahl der reingeschmissenen messages erkennt und erst eine abarbeitet und die nächste erst startet sobald eine message aus deiner node ausgegeben wurde? Das wäre godlike, spiele aktuell mit ner statustime von 5000 ms und lasse über nen loop die 2. message nach 5250 ms reinschmeißen. Wenn er es automatisch erkennt wäre das natürlich geil.

Andere alternative wäre z.B. eine msg mit folgendem inhalt reinknallen:

msg.payload: {statustime: 5, disabletime: 900, pihole: all}

Deine Node erkannt, wenn all auf msg.payload.pihole eingeht, dass er ALLE configurierten piholes den befehl übermittelt und mit nem delay von z.B. 10 ms die status messages rausknallt oder gar "sammelt" und dann in einer msg rauswirft also am besten in einem array, so könnte man die abfrage mit ner for schleife in einer function node danach machen :D

Unnnd falls das too much wird und man einfach es jetzt so lassen sollte wie es ist, ist das auch iO. Bin son kleiner Perfektionist... werde immer was finden was man optimieren verbessern etc. könnte, aber irgendwann is natürlich auch genug gell ;)

naimo84 commented 4 years ago

A gutes neues 😉 Ich denke, beides müsste möglich sein. Variante 2 gefällt mir persönlich aber fast besser. Und dürfte auch deutlich einfacher umzusetzen sein 😀

Bei Variante 1 sehe ich gerade noch keinen Unterschied zu der bisherigen Lösung. Node-red sollte das eig. genauso regeln.

nicedevil007 commented 4 years ago

Variante 1 funzt definitiv net ;)

Ich schmeiß 2 verschiedene msg mit nem delay von z.b. 250 ms rein und deine Node kommt dann leider net hinterher und verwechselt den namen vom pihole teilweise (siehe auch das bereits geclosete issue).

Aber wenn Variante 2 klappt, dann is das alles eh obsolet :D Könntest da ja sogar nen Haken in die confignode reinmachen, so dass man anklicken kann "immer Alle abfragen" oder sowas :)

naimo84 commented 4 years ago

Mit v0.1.4 sollte nun beide Varianten funktionieren 😉

Variante 1 hätte theoretisch vorher schon funktioniert... wenn ich's richtig programmiert hätte 😉