FHEM Forum

FHEM - Hardware => FRITZ!Box => Thema gestartet von: betateilchen am 15 November 2017, 22:23:32

Titel: [bug] Passwort wird nicht gelöscht
Beitrag von: betateilchen am 15 November 2017, 22:23:32
Beim Löschen eines fritzbox-devices wird ein ggf. in FHEM gespeichertes Passwort zu diesem device nicht gelöscht.
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: rudolfkoenig am 16 November 2017, 09:07:43
Was fuer ein Typ ist dein Fritzbox-Device? Mir sind 3 (bzw. 4, wenn man FBAHA dazuzaehlt) unterschiedliche, mit jeweils unterschiedlichen Maintainer bekannt.
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: rudolfkoenig am 16 November 2017, 09:49:41
FBAHAHTTP loescht ab sofort das gespeicherte Passwort nach einem delete.
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: nils_ am 16 November 2017, 09:54:51
vmtl. ähnliche Konstellation:
https://svn.fhem.de/trac/browser/trunk/fhem/FHEM/72_FRITZBOX.pm#L4645
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: betateilchen am 16 November 2017, 15:39:57
Zitat von: rudolfkoenig am 16 November 2017, 09:07:43
Was fuer ein Typ ist dein Fritzbox-Device?

FRITZBOX, Maintainer tupol
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: CoolTux am 16 November 2017, 15:48:02
Ich bin Mal verwegen und behaupte das keiner die Passwörter aus dem FHEM Container löscht  ;D



Grüße
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: marvin78 am 16 November 2017, 15:59:17
Wofür gibt es X_Delete?
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: CoolTux am 16 November 2017, 17:21:54
Wenn man sowas nicht explizit im Kopf behält verliert man das schnell Mal beim coden aus dem Augen.
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: betateilchen am 16 November 2017, 20:24:36
Zitat von: CoolTux am 16 November 2017, 15:48:02
Ich bin Mal verwegen und behaupte das keiner die Passwörter aus dem FHEM Container löscht

Eine diesbezügliche Wette würdest Du verlieren, wenn Du die Aussage auf alle Module beziehst.


sub GDS_Delete() {
my ($hash, $arg) = @_;
my $name = $hash->{NAME};
setKeyValue($name."_user",undef);
setKeyValue($name."_pass",undef);
setKeyValue($name."_host",undef);
}
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: CoolTux am 16 November 2017, 20:26:12
Zitat von: betateilchen am 16 November 2017, 20:24:36
Eine diesbezügliche Wette würdest Du verlieren, wenn Du die Aussage auf alle Module beziehst.


sub GDS_Delete() {
my ($hash, $arg) = @_;
my $name = $hash->{NAME};
setKeyValue($name."_user",undef);
setKeyValue($name."_pass",undef);
setKeyValue($name."_host",undef);
}


Streber  :P ;D
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: Markus Bloch am 19 November 2017, 09:20:39
Zitat von: rudolfkoenig am 16 November 2017, 09:49:41
FBAHAHTTP loescht ab sofort das gespeicherte Passwort nach einem delete.

dito für FB_CALLMONITOR.
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: tupol am 19 November 2017, 13:32:20
fixed für FRITZBOX

Beim Rename geht es nicht, weil der Routine laut wiki kein $hash übergeben wird.
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: Markus Bloch am 19 November 2017, 13:42:54
Zitat von: tupol am 19 November 2017, 13:32:20
Beim Rename geht es nicht, weil der Routine laut wiki kein $hash übergeben wird.

Geht durchaus:

sub X_Rename($$)
{
    my ($new, $old) = @_; 
   
    my $hash = $defs{$new};
   
...


Macht aber kein Sinn, da die Hash-Referenz die gleiche bleibt. Der Hash wird 1:1 mitgenommen. Zum Zeitpunkt des Aufrufes von der RenameFn existiert $defs{$old} bereits nicht mehr.
Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: Markus Bloch am 19 November 2017, 14:01:16
Beispielhaft so in deinem Fall:


sub FRITZBOX_Rename($$)
{
    my ($new, $old) = @_; 
   
    my $old_index = "FRITZBOX_".$old."_passwd";
    my $new_index = "FRITZBOX_".$new."_passwd";
   
    my ($err, $old_pwd) = getKeyValue($old_index);
   
    setKeyValue($new_index, $old_pwd);
    setKeyValue($old_index, undef);
}

Titel: Antw:[bug] Passwort wird nicht gelöscht
Beitrag von: tupol am 19 November 2017, 14:35:36
Danke.