FHEM Forum

FHEM => Automatisierung => Thema gestartet von: rudolfkoenig am 23 Dezember 2017, 20:32:26

Titel: SecurityCheck cleanup
Beitrag von: rudolfkoenig am 23 Dezember 2017, 20:32:26
Ich habe die als motd angezeigte SecurityCheck Pruefung bzw. die angezeigten Texte ueberarbeitet:
- die Pruefung ist aus den einzelnen Modulen (bisher nur FHEMWEB und telnet) nach fhem.pl gewandert. Falls weitere Module eine Pruefung haben wollen, dann bitte mich ansprechen, dann werde ich dafuer eine Loesung finden, was die betroffenen Modulnamen nicht hart kodiert.
- In der motd wird vorgeschlagen, ein allowed Instanz anzulegen. Falls jemand ein Alternativ-allowed baut, mich ebenfalls ansprechen, damit der Endanwender die Wahl hat.
- Ueber FHEMWEB ist die Passwort-Vergabe ohne Doku lesen moeglich, siehe Anhang.

Ich habe jetzt diverse Kombinationen getestet, habe aber vermutlich trotzdem irgendetwas uebersehen, bitte sowas hier melden.
Titel: Antw:SecurityCheck cleanup
Beitrag von: DeeSPe am 25 Dezember 2017, 11:50:17
Ich habe eine Warnung im Log:

PERL WARNING: Use of uninitialized value in pattern match (m//) at fhem.pl line 5383.

Gruß
Dan
Titel: Antw:SecurityCheck cleanup
Beitrag von: betateilchen am 25 Dezember 2017, 11:53:13
Das passiert vermutlich immer dann, wenn es überhaupt kein motd in global gibt.

Vermutlich könnte AttrVal('global','motd','') anstatt des direkten Zugriffs auf %attr das Problem schon lösen.
Titel: Antw:SecurityCheck cleanup
Beitrag von: DeeSPe am 25 Dezember 2017, 11:54:22
Das scheint mir auch so.
Besser wäre wohl:
} elsif($attr{global}{motd} && $attr{global}{motd} =~ m/^SecurityCheck/) {

Gruß
Dan
Titel: Antw:SecurityCheck cleanup
Beitrag von: betateilchen am 25 Dezember 2017, 11:56:07
ich habe eben noch einen Vorschlag ergänzt, ohne direkt auf %attr zugreifen zu müssen.
Titel: Antw:SecurityCheck cleanup
Beitrag von: DeeSPe am 25 Dezember 2017, 11:58:25
Hehe, die Idee ist mir auch gerade gekommen.

Gruß
Dan
Titel: Antw:SecurityCheck cleanup
Beitrag von: rudolfkoenig am 25 Dezember 2017, 17:48:56
Danke fuer den Hinweis, habs auf AttrVal umgebaut.
Titel: Antw:SecurityCheck cleanup
Beitrag von: krikan am 26 Dezember 2017, 21:48:53
Hat diese Aenderung eventuell zu diesem für mich nachvollziehbaren Problem geführt: https://forum.fhem.de/index.php/topic,81612.0.html ?