mschlenstedt / loxberry-plugin-nuki Goto Github PK

Optional=0 ist für Lock
Für Opener muss der Typ mitgegeben werden.

lockAction benötigt auch den deviceType für den Opener im Template, haben wir aber schon drin.

Wenn ich den Input/Output Popup öffne, wird er zunächst schmal geöffnet und nach laden dann verbreitert. Dabei springt der Bildschirm "unschön" hin und her.

Das Gleiche beim Bridge-Details-Popup, wobei es hier nur minimal ist.

-> On the Fly Inhalt im Popup auf max-width beschränken.
-> Eventuell Schrift im Popup auch etwas kleiner (wirkt auf meinem Laptop sehr groß)

Das checkonline (führt /info ohne Token aus) ist überflüssig, weil checktoken (/info mit Token) das gleiche macht.

Es reicht ein checktoken. Das Ergebnis ist dann entweder

• nicht erreichbar (Online rot, Token schwarz)
• http 401 (Online grün, Token rot)
• Response (Online grün, Token grün)

Es behebt den von uns erwarteten Error im Log.
Das geht dann auch schneller.

Irgendein Fehlerhandling muss noch kommen, wenn die Callback-Pflege seinen Callback anlegen will, aber die Liste der drei Callbacks bereits voll ist.
Derzeit gibt die Funktion auf und nichts weiter.

• Fehler korrekt handlen (entsprechenden Errorcode im callback_add verarbeiten)
• Notify

Wenn eine Bridge authentifiziert wird, soll automatisch nach neuen Devices gesucht werden.

Bei Requests und Responses
Am besten maskieren (L*****)

Die Ladezeit des SecurePIN-Checks ist zu hoch, weil schon so viel Code und Libs geladen sind.

-> eigenes .cgi nur mit LB:System und dem SecurePIN-Teil, ohne andere Libs. Response mit hardcodetem json statt JSON-Libs.

Mir kommt generell die Ladezeit der Forms langsam vor.

Es muss noch geprüft werden, ob alle AJAX-Calls mit SecurePIN geschützt sind.

Scheint bei Umlauten ("Haustür") sonst nicht mit allen Browsern zu funktionieren.
https://www.loxforum.com/forum/projektforen/loxberry/plugins/210944-nuki-smartlock-plugin?p=219778#post219778

Dateinamen:

Möglichkeit alte oder falsche Callbacks zu löschen. Editieren soll explizit nicht möglich sein.

Beim Discovery muss auch die discoveryBridgeId geprüft werden, und wenn vorhanden, die Daten der bestehenden Bridge aktualisiert.

Fehlen noch komplett

WARNUNG NUKI SmartLock: HARDCODED PATH'S: Das Plugin nutzt einen hardkodierten Pfad zu /opt/loxberry. Dies könnte zu Problemen führen und sollte vom Plugin-Author behoben werden: [email protected]

https://github.com/mschlenstedt/LoxBerry-Plugin-Nuki/search?q=opt%2Floxberry&unscoped_q=opt%2Floxberry

Wenn die /Auth-Funktion fehlschlägt, gibt es eine entsprechende Meldung im GUI (z. B. wenn das Anlegen des Callbacks fehlschlägt).

Die BridgeID soll nicht in Klammern angezeigt werden.

Zudem: Welche BridgeID nehmen wir jetzt im WebIf? Hardware? Discovery? Eigene?

Wenn man eine Bridge manuell hinzufügt, muss die BridgeID anschließend ausgelesen werden, sonst kann bei einer falschen BridgeID die Callback-Funktion fehlschlagen.

Aktuell wird die eingegebene BridgeID nicht überprüft. Zudem ist die in der app angezeigte BridgeID unterschiedlich zur BridgeID, die per JSON zurückkommt!

Ich habe Dummy-Icons im Code, die noch befüllt/korrigiert werden müssen:

• nuki-unknown.svg --> DEVICETYPE unknown
• devices_vo_download_32.png --> Action-Symbol bei Devices für "Virtual Inputs / Virtual Outputs"

Bei Status-Abfragen direkt vom Keyturner soll ein Retry stattfinden, wenn die Bridge das Gerät nicht erreichen konnte.
https://www.loxforum.com/forum/projektforen/loxberry/plugins/210944-nuki-smartlock-plugin?p=247855#post247855

Unter -80 ist schlechte Verbindung (Rot)
https://developer.nuki.io/t/rssi-value-boundaries-and-green-yellow-red/2532/2?u=christiantf

Der stündliche Cronjob soll nicht nur die Callbacks prüfen, sondern auch einen Statusabruf bei den Bridges durchführen, und senden.

https://github.com/as19git67/nukible

Index.cgi:
Es muss erkannt werden, ob der Aufruf von CLI oder Apache kommt.
Jeder Apache Call muss vorher den SecPin prüfen.

Settings.html:
Jeder Ajax-Call muss implizit den SecPin mitsenden.

Security Issue ist, dass Ajax-Calls teilweise ohne SecPin funktionieren.

Das hat in der Arbeit anders ausgesehen als daheim. Der Link soll unter die Checkbox

Das er beim Callback den Hostnamen nimmt zerschiesst zumindest bei mir die Konfiguration.

Ich hab den Hostnamen im LoxBerry WebIf nicht geändert (ist folglich per Default "loxberry"). In der Fritte habe ich aber dem LoxBerry den Namen "loxberrydevil" zugewiesen.

Der Callback zeigt jetzt auf http://loxberry:80/plugins/nukismartlock/callback.php?lbuid=ouXDU und geht damit ins Leere...

Keine Ahnung, aber ich mach das mit der Fritte immer so... Vielleicht ist das ein genereller Bug im LoxBerry. Er müsste seinen Hostnamen eigentlich per DNS abfragen und dann selbst korrekt setzen. Oder per DHCP (kriegt er den mitgeliefert?) setzen. So ist es auf jeden Fall bei mir zerschossen.

Feature für MQTT-Gateway >= 0.8.3
https://www.loxwiki.eu/display/LOXBERRY/MQTT+Gateway+for+plugin+developers#MQTTGatewayforplugindevelopers-Injectsubscriptions,conversion,resetAfterSendbyyourplugin

• Funktion zum Auslesen des Bridge Logs, am besten per HTTP-Trigger (Zeitplan am MS)
• Die Daten oder irgendein Timestamp muss am LB gespeichert werden, um neue Logeinträge erkennen zu können
• Die Anzahl neuer Log-Einträge seit dem letzten Call soll jedenfalls gepublished werden
• Evt. auch den letzten Log-Eintrag (oder eine definierte Anzahl an Log-Einträgen) mit Zeit und Text
• Evt. ein Summary der neuen Log-Einträge

Endpoint: /log

Example-Response | [  {"timestamp": "2018-10-06T16:46:05+00:00", "deviceType": “...” },  {"timestamp": "2018-10-06T16:46:05+00:00", "deviceType": “...” }, …]

https://developer.nuki.io/page/nuki-bridge-http-api-1-12/4#heading--log

Mit irgendeinem jQm-Tag kann man den Popups einen Close-Button verpassen.
Damit erschließt es sich leichter, wie man das Popup wieder schließt.

Zur Überwachung von Callbacks

https://www.loxforum.com/forum/projektforen/loxberry/plugins/210944-nuki-smartlock-plugin?p=302079#post302079

...muss noch integriert werden. Code vom MiRobot-Plugin: https://github.com/mschlenstedt/LoxBerry-Plugin-MiRobot2Lox-NG

PHP Skript anpassen für mehrere Bridges/Devices und zum Lesen der Config aus den Configfiles.

Ich habe hier

Geöffnet werden kann das Popup derzeit durch Klick auf die Bridge-ID, es steht aber noch nichts drin.

Hier können Details wie Firmware-Version usw. angezeigt werden, wichtig wäre mir jedoch die Liste der Callbacks und eine Lösch-Funktion (Button) der Callbacks.
Funktionen zum Listen und Löschen gibt es schon im index.cgi (aber noch ohne ajax=... Aufruf-Funktion), und der Response der Funktionen ist noch undefiniert.

Use-Case bei der Callback-Liste ist, dass man alte, falsche oder fremde Callbacks löschen kann. Ein Hinzufügen oder Editieren der Callbacks ist explizit nicht gewünscht.

Aktuell noch Festwerte: https://github.com/mschlenstedt/LoxBerry-Plugin-Nuki/blob/master/webfrontend/html/callback.php

Ist im Prinzip das gleiche.

Wenn eine Bridge aus dem WebIf gelöscht wird, soll auch der Callback gelöscht werden.

Zudem sollen auch alle zugehörigen Devices zur Bridge aus der Config entfernt werden (sollte gehen, indem "searchdevices" einfach erneut aufgerifen wird).

...fehlen noch

Wenn ein Wert nicht definiert ist, soll das Feld im WebIf leer sein und nicht "undefined" angezeigt werden.

Devices-Übersicht ist in Ordnung. Vermultich ein MarkUp-Fehler...