Bugreport & PAtCH: DbLog loggt nicht mehr seit Update v2.21.2 von einer 2016er

[ska-]

In der FAQ steht zwar "schreib einen bugreport", aber nicht wie/wo. Wenn das hier der falsche Ort ist, bitte verschieben und FAQ aktualisieren. Danke.

Nach dem Update von einer 2016er Version von DbLog auf v2.21.2 habe ich keine neuen Werte mehr in der DB,  kann aber alte abrufen. Grund ist, dass ich eine Passwort-lose Authentizierung verwende:

Code Auswählen Erweitern

%dbconfig= (
        connection => "Pg:database=fhem",
        user => "fhem",
#        password => "fhempassword"
);

Fehler:

Code Auswählen Erweitern

DbLog logdb - Error while reading ./db.conf: 'could not read password'
1: PERL WARNING: Use of uninitialized value in string eq at ./FHEM/93_DbLog.pm line 2144.
1: PERL WARNING: Use of uninitialized value in string eq at ./FHEM/93_DbLog.pm line 2312.
1: PERL WARNING: Use of uninitialized value in string eq at ./FHEM/93_DbLog.pm line 2320.
1: PERL WARNING: Use of uninitialized value in string eq at ./FHEM/93_DbLog.pm line 2327.
1: PERL WARNING: Use of uninitialized value in string eq at ./FHEM/93_DbLog.pm line 2334.
PERL WARNING: Use of uninitialized value in pattern match (m//) at ./FHEM/93_DbLog.pm line 394.

Es wird trotz der Fehler eine Verbindung mit dem Postgres-Server als Nutzer fhem aufgebaut, es werden auch *alte* Werte zurueck gegeben (in Plots bspw.), aber es gelangen keine neuen Werte in die DB.

Deshalb bitte folgenden Patch einspielen:

Code Auswählen Erweitern

--- fhem_nachUpgrade2/FHEM/93_DbLog.pm 2017-07-22 22:16:18.762407331 +0200
+++ fhem/./FHEM/93_DbLog.pm 2017-07-23 09:45:57.074719737 +0200
@@ -2088,6 +2088,12 @@
   $hash->{dbconn} = $dbconfig{connection};
-  return "could not read user" if (!defined $dbconfig{user});
+  if(defined $dbconfig{user}) {
   $hash->{dbuser} = $dbconfig{user};
-  return "could not read password" if (!defined $dbconfig{password});
+  } else {
+    Log3 $hash->{NAME}, 1, "could not read DB user from file $configfilename.";
+  }
+  if(defined $dbconfig{password}) {
   $attr{"sec$name"}{secret} = $dbconfig{password};
+  } else {
+    Log3 $hash->{NAME}, 1, "could not read DB password from file $configfilename.";
+  }



Damit erhaelt man eine Warnung, wenn Nutzername oder Passwort fehlen, aber MODEL wird gesetzt. Wenn die DB-Verbindung weder Nutzername noch Passwort braucht, bspw. durch implizite Authentifizierung, ist die Konfiguration des Hashs komplett. Wenn man beides braucht, gelingt ohnehin keine DB-Verbindung, aber man bekommt die Fehler-Logs und kann agieren.

[CoolTux]

Hallo Ska,

Vielen Dank für Deinen Patch.
Welcher Forenbereich für das entsprechende Modul Verwendung findet kannst Du in der Datei MAINTAINER nach lesen. Ich habe jetzt nicht geschaut, aber sollte das hier das falsche Forum sein, kannst Du Deinen Beitrag einfach ins richtige Forum verschieben. Ganz unten Links "verschieben"

[DS_Starter]

Hallo Ska,

danke für die Info. Baue ich demnächst mit ein.

Ansonsten ist dieses Forum richtig für DbLog:     FHEM Forum » FHEM » Automatisierung

Grüße
Heiko

[DS_Starter]

@Ska,

Frage... reicht es nicht wenn du deine cfg so aufbauen würdest:

Code Auswählen Erweitern


%dbconfig= (
        connection => "Pg:database=fhem",
        user => "fhem",
        password => ""
);


Dann würde DbLog in der jetzigen Form problemlos funktionieren.
Bitte teste diese Konfiguration.

Danke,
Heiko

[dev0]

Baue ich demnächst mit ein.

Sollte man das wirklich machen? Ich fände aus "sicherheitstechnischen Gründen" das nicht sinnvoll.

[DS_Starter]

Sollte man das wirklich machen? Ich fände aus "sicherheitstechnischen Gründen" das nicht sinnvoll.

Ich prüfe es vorher natürlich noch und baue es nicht blind ein.
Aber davon abgesehen sollte mein Hinweis:

Code Auswählen Erweitern


password => ""


funktionieren (wie bei SQLite) und keine Codeänderung erfordern.

Schönen Sonntag noch und LG,
Heiko

[dev0]

Ich prüfe es vorher natürlich noch und baue es nicht blind ein.

Davon bin ich auch nicht ausgegangen.

Ich wollte nur kundtun, dass es mMn eine Sicherheitslücke ist, wenn man ohne Authentifizierung eine DB beschreiben kann, die Daten weiterverarbeitet.

[ska-]

@Ska,

Frage... reicht es nicht wenn du deine cfg so aufbauen würdest:

Code Auswählen Erweitern


%dbconfig= (
        connection => "Pg:database=fhem",
        user => "fhem",
        password => ""
);


Dann würde DbLog in der jetzigen Form problemlos funktionieren.
Bitte teste diese Konfiguration.

Danke,
Heiko

Ja, klappt so. Sollte dann bitte in der db.conf dokumentiert werden.

[ska-]

Davon bin ich auch nicht ausgegangen.

Ich wollte nur kundtun, dass es mMn eine Sicherheitslücke ist, wenn man ohne Authentifizierung eine DB beschreiben kann, die Daten weiterverarbeitet.

Die Datenbank legt fest, welche Authentifizierung sie akzeptiert und damit die Sicherheitsstufe.

Wieviel sicherer ist eine Authentifizierung mit Passwort ueber eine lokale Socket, wenn das Passwort im Klartext auf dem selben Rechner gespeichert ist? Wenn es darueberhinaus genau ein Person gibt, die sich auf dem Server einloggen darf. Mal davon abgesehen, wenn man (der potentielle Angreifer) schon lokal auf dem Rechner ist, ... .

[DS_Starter]

Ja, klappt so. Sollte dann bitte in der db.conf dokumentiert werden.

Danke für die Rückinfo. Ja, werde ich ergänzen damit man nicht darüber stolpert.

VG

[dev0]

@ska-: Es gibt hier viele User, die von Security, lokalen Sockets, etc. keine Ahnung haben und einfach nur froh sind, wenn sie es irgendwie zum Laufen bringen und ein FHEM betreiben, dass nach außen offen wie ein Scheunentor ist. Ein Passwortzwang wäre mMn in diesen Fällen nutzlich.