Improve log on 2F unregistration (#1702)
This commit is contained in:
parent
ea1609819a
commit
5512d38f81
|
@ -301,12 +301,18 @@ sub run {
|
||||||
}
|
}
|
||||||
|
|
||||||
# Delete TOTP 2F device
|
# Delete TOTP 2F device
|
||||||
|
my $TOTPName;
|
||||||
|
foreach (@$_2fDevices) {
|
||||||
|
$TOTPName = $_->{name} if $_->{epoch} eq $epoch;
|
||||||
|
}
|
||||||
@$_2fDevices = grep { $_->{epoch} ne $epoch } @$_2fDevices;
|
@$_2fDevices = grep { $_->{epoch} ne $epoch } @$_2fDevices;
|
||||||
$self->logger->debug(
|
$self->logger->debug(
|
||||||
"Delete 2F Device : { type => 'TOTP', epoch => $epoch }");
|
"Delete 2F Device : { type => 'TOTP', epoch => $epoch, name => $TOTPName }"
|
||||||
|
);
|
||||||
$self->p->updatePersistentSession( $req,
|
$self->p->updatePersistentSession( $req,
|
||||||
{ _2fDevices => to_json($_2fDevices) } );
|
{ _2fDevices => to_json($_2fDevices) } );
|
||||||
$self->userLogger->notice('TOTP deletion succeed');
|
$self->userLogger->notice(
|
||||||
|
"TOTP $TOTPName unregistration succeeds for $user");
|
||||||
return [
|
return [
|
||||||
200,
|
200,
|
||||||
[ 'Content-Type' => 'application/json', 'Content-Length' => 12, ],
|
[ 'Content-Type' => 'application/json', 'Content-Length' => 12, ],
|
||||||
|
|
|
@ -288,12 +288,18 @@ sub run {
|
||||||
}
|
}
|
||||||
|
|
||||||
# Delete U2F device
|
# Delete U2F device
|
||||||
|
my $keyName;
|
||||||
|
foreach (@$_2fDevices) {
|
||||||
|
$keyName = $_->{name} if $_->{epoch} eq $epoch;
|
||||||
|
}
|
||||||
@$_2fDevices = grep { $_->{epoch} ne $epoch } @$_2fDevices;
|
@$_2fDevices = grep { $_->{epoch} ne $epoch } @$_2fDevices;
|
||||||
$self->logger->debug(
|
$self->logger->debug(
|
||||||
"Delete 2F Device : { type => 'U2F', epoch => $epoch }");
|
"Delete 2F Device : { type => 'U2F', epoch => $epoch, name => $keyName }"
|
||||||
|
);
|
||||||
$self->p->updatePersistentSession( $req,
|
$self->p->updatePersistentSession( $req,
|
||||||
{ _2fDevices => to_json($_2fDevices) } );
|
{ _2fDevices => to_json($_2fDevices) } );
|
||||||
$self->userLogger->notice('U2F key unregistration succeed');
|
$self->userLogger->notice(
|
||||||
|
"U2F key $keyName unregistration succeeds for $user");
|
||||||
return [
|
return [
|
||||||
200,
|
200,
|
||||||
[ 'Content-Type' => 'application/json', 'Content-Length' => 12, ],
|
[ 'Content-Type' => 'application/json', 'Content-Length' => 12, ],
|
||||||
|
|
|
@ -181,12 +181,18 @@ sub run {
|
||||||
}
|
}
|
||||||
|
|
||||||
# Delete Yubikey device
|
# Delete Yubikey device
|
||||||
|
my $UBKName;
|
||||||
|
foreach (@$_2fDevices) {
|
||||||
|
$UBKName = $_->{name} if $_->{epoch} eq $epoch;
|
||||||
|
}
|
||||||
@$_2fDevices = grep { $_->{epoch} ne $epoch } @$_2fDevices;
|
@$_2fDevices = grep { $_->{epoch} ne $epoch } @$_2fDevices;
|
||||||
$self->logger->debug(
|
$self->logger->debug(
|
||||||
"Delete 2F Device : { type => 'UBK', epoch => $epoch }");
|
"Delete 2F Device : { type => 'UBK', epoch => $epoch, name => $UBKName }"
|
||||||
|
);
|
||||||
$self->p->updatePersistentSession( $req,
|
$self->p->updatePersistentSession( $req,
|
||||||
{ _2fDevices => to_json($_2fDevices) } );
|
{ _2fDevices => to_json($_2fDevices) } );
|
||||||
$self->userLogger->notice('Yubikey deletion succeed');
|
$self->userLogger->notice(
|
||||||
|
"Yubikey $UBKName unregistration succeeds for $user");
|
||||||
return [
|
return [
|
||||||
200,
|
200,
|
||||||
[ 'Content-Type' => 'application/json', 'Content-Length' => 12, ],
|
[ 'Content-Type' => 'application/json', 'Content-Length' => 12, ],
|
||||||
|
|
Loading…
Reference in New Issue
Block a user