93_dblog keine plausibilitätsprüfung bei cmd parametern

Begonnen von chris1284, 09 März 2017, 07:12:05

Vorheriges Thema - Nächstes Thema

chris1284

Hallo,

musste gerade feststellen das dblog sich nicht darum kümmert ob set reducelog (wharscheinlich ehr alle cmds parameter ) sinn ergeben und prüft nicht auf ihre korrektheit .

ich wollte ein reducelog 30 average ausführen und habe ein reducelog 30 verage geschrieben. das modul läuft dann dummerweise munter los und mach nur den reducelog 30 und haut somit alle daten weg obwohl der befehl / parameter 2 falsch ist. hier sollt schon geprüft werden ob die parameter sinn eregeben, ist ja nicht so ungefährlich wie ein set lampe ofkjhdjdf wo maximal nichts geschaltet wird. hier wird richtig "schaden" angerichtet

DS_Starter

Hallo Chris,

guter Hinweis. Ich hatte bei der Weiterentwicklung von DbLog die Reducelog-Funktion wie sie war übernommen und nicht geändert außer was bezüglich der DB-Handle nötig war bzw. der nonblocking-Gestaltung.
Ich sehe mal einen verbesserten Plausi-check beim nächsten Update mit vor.
Danke für den Hinweis !

Gruß
Heiko
Proxmox+Debian+MariaDB, PV: SMA, Victron MPII+Pylontech+CerboGX
Maintainer: SSCam, SSChatBot, SSCal, SSFile, DbLog/DbRep, Log2Syslog, SolarForecast,Watches, Dashboard, PylonLowVoltage
Kaffeekasse: https://www.paypal.me/HMaaz
Contrib: https://svn.fhem.de/trac/browser/trunk/fhem/contrib/DS_Starter