SecurityCheck cleanup

[rudolfkoenig]

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.

[DeeSPe]

Ich habe eine Warnung im Log:

Code Auswählen Erweitern

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

Gruß
Dan

[betateilchen]

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.

[DeeSPe]

Das scheint mir auch so.
Besser wäre wohl:

Code Auswählen Erweitern

} elsif($attr{global}{motd} && $attr{global}{motd} =~ m/^SecurityCheck/) {

Gruß
Dan

[betateilchen]

ich habe eben noch einen Vorschlag ergänzt, ohne direkt auf %attr zugreifen zu müssen.

[DeeSPe]

Hehe, die Idee ist mir auch gerade gekommen.

Gruß
Dan

[rudolfkoenig]

Danke fuer den Hinweis, habs auf AttrVal umgebaut.

[krikan]

Hat diese Aenderung eventuell zu diesem für mich nachvollziehbaren Problem geführt: https://forum.fhem.de/index.php/topic,81612.0.html ?