From 9536c4dcd43fdd07f884e6fbfdf0a7bba0a36bb0 Mon Sep 17 00:00:00 2001 From: Wolfgang Bumiller Date: Tue, 2 Apr 2019 12:21:54 +0200 Subject: [PATCH] delete TFA entries when deleting a user Signed-off-by: Wolfgang Bumiller --- PVE/API2/User.pm | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/PVE/API2/User.pm b/PVE/API2/User.pm index 4c859dc..4458fc1 100644 --- a/PVE/API2/User.pm +++ b/PVE/API2/User.pm @@ -355,11 +355,14 @@ __PACKAGE__->register_method ({ $plugin->delete_user($cfg, $realm, $ruid); } + # Remove TFA data before removing the user entry as the user entry tells us whether + # we need ot update priv/tfa.cfg. + PVE::AccessControl::user_set_tfa($userid, $realm, undef, undef, $usercfg, $domain_cfg); + delete $usercfg->{users}->{$userid}; PVE::AccessControl::delete_user_group($userid, $usercfg); PVE::AccessControl::delete_user_acl($userid, $usercfg); - cfs_write_file("user.cfg", $usercfg); }, "delete user failed"); -- 2.39.2