smarthome.... http Zähler

[okaegi]

Korrektur Faktor vom http Zähler für Smarthome....

[benderl]

Ich habe leider den ursprünglichen PR nicht gesehen, daher haben sich jetzt ein paar Sicherheitsprobleme eingeschlichen.

• Der Inhalt des Parameter "d" muss validiert werden.
• Der Inhalt der Datei muss validiert werden.
• Im Fehlerfall muss der HTTP-Header entsprechend gesetzt werden; z.B. auf 4xx, wenn die Datei nicht existiert oder der Parameter ungültig ist, 5xx für andere Probleme mit dem Inhalt

Dann frage ich mich, welchen Sinn die PHP-Skripte haben. Sowohl bezug.php als auch leistung.php geben lediglich einen Dateiinhalt aus der Ramdisk zurück. Warum also nicht gleich diesen Pfad für die URL des HTTP-Zählers verwenden? Die Skripte bieten keinen Mehrwert, sondern nur potentielle Sicherheitsprobleme.

[okaegi]

Salü Lutz,
Mit diesen php scrips können die Smarthomedevices auf der openwb 2.0 Haupseite angezeigt werden.
Siehe https://forum.openwb.de/viewtopic.php?t=9712
Diese Dateien werden intern vom Smarthome mit der aktuellen Leistung und dem aktuellen Zählerstand geschrieben.
Hast du mit ggf ein Beispiel für die vehrschiedenen Error Messages ?
Gruss Oliver

[benderl]

Salü Lutz, Mit diesen php scrips können die Smarthomedevices auf der openwb 2.0 Haupseite angezeigt werden. Siehe https://forum.openwb.de/viewtopic.php?t=9712 Diese Dateien werden intern vom Smarthome mit der aktuellen Leistung und dem aktuellen Zählerstand geschrieben.

Das ist mir schon klar, habe den Beitrag im Forum gesehen. Welchen Vorteil bietet das jetzt im Gegensatz zur URL http://<IP>/openWB/ramdisk/device<number>_wh? Ich sehe da keinen Mehrwert.

Hast du mit ggf ein Beispiel für die vehrschiedenen Error Messages ? Gruss Oliver

Du könntest Dir runs/http-api/v1/index.php ansehen. Davor sollte aber geklärt werden, ob die PHP-Dateien überhaupt notwendig sind.

[okaegi]

Salü Lutz,
Du hast recht (, ich habe den Forumsbeitrag angepasst) und schliesse den PR. Es braucht keinen Pr.
https://forum.openwb.de/viewtopic.php?p=122027#p122027
Gruss Oliver

[okaegi]

bessere Lösung gefunden nur über paraemtrisierung