Remove 2FA backup codes on user deletion
Signed-off-by: Joas Schilling <coding@schilljs.com>
This commit is contained in:
parent
c56c98183d
commit
cd79eb1b2c
|
@ -1,7 +1,8 @@
|
||||||
<?php
|
<?php
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @author Christoph Wurst <christoph@winzerhof-wurst.at>
|
* @copyright Copyright (c) 2017 Joas Schilling <coding@schilljs.com>
|
||||||
|
*
|
||||||
|
* @author Joas Schilling <coding@schilljs.com>
|
||||||
*
|
*
|
||||||
* @license GNU AGPL version 3 or any later version
|
* @license GNU AGPL version 3 or any later version
|
||||||
*
|
*
|
||||||
|
@ -20,6 +21,5 @@
|
||||||
*
|
*
|
||||||
*/
|
*/
|
||||||
|
|
||||||
// @codeCoverageIgnoreStart
|
$app = new \OCA\TwoFactorBackupCodes\AppInfo\Application();
|
||||||
OC_App::registerPersonal('twofactor_backupcodes', 'settings/personal');
|
$app->register();
|
||||||
// @codeCoverageIgnoreEnd
|
|
||||||
|
|
|
@ -0,0 +1,62 @@
|
||||||
|
<?php
|
||||||
|
/**
|
||||||
|
* @copyright Copyright (c) 2017 Joas Schilling <coding@schilljs.com>
|
||||||
|
*
|
||||||
|
* @author Joas Schilling <coding@schilljs.com>
|
||||||
|
*
|
||||||
|
* @license GNU AGPL version 3 or any later version
|
||||||
|
*
|
||||||
|
* This program is free software: you can redistribute it and/or modify
|
||||||
|
* it under the terms of the GNU Affero General Public License as
|
||||||
|
* published by the Free Software Foundation, either version 3 of the
|
||||||
|
* License, or (at your option) any later version.
|
||||||
|
*
|
||||||
|
* This program is distributed in the hope that it will be useful,
|
||||||
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||||
|
* GNU Affero General Public License for more details.
|
||||||
|
*
|
||||||
|
* You should have received a copy of the GNU Affero General Public License
|
||||||
|
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
*
|
||||||
|
*/
|
||||||
|
|
||||||
|
namespace OCA\TwoFactorBackupCodes\AppInfo;
|
||||||
|
|
||||||
|
use OCA\TwoFactorBackupCodes\Db\BackupCodeMapper;
|
||||||
|
use OCP\AppFramework\App;
|
||||||
|
use OCP\Util;
|
||||||
|
|
||||||
|
class Application extends App {
|
||||||
|
public function __construct () {
|
||||||
|
parent::__construct('twofactor_backupcodes');
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Register the different app parts
|
||||||
|
*/
|
||||||
|
public function register() {
|
||||||
|
$this->registerHooksAndEvents();
|
||||||
|
$this->registerPersonalPage();
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Register the hooks and events
|
||||||
|
*/
|
||||||
|
public function registerHooksAndEvents() {
|
||||||
|
Util::connectHook('OC_User', 'post_deleteUser', $this, 'deleteUser');
|
||||||
|
}
|
||||||
|
|
||||||
|
public function deleteUser($params) {
|
||||||
|
/** @var BackupCodeMapper $mapper */
|
||||||
|
$mapper = $this->getContainer()->query(BackupCodeMapper::class);
|
||||||
|
$mapper->deleteCodesByUserId($params['uid']);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Register personal settings for notifications and emails
|
||||||
|
*/
|
||||||
|
public function registerPersonalPage() {
|
||||||
|
\OCP\App::registerPersonal($this->getContainer()->getAppName(), 'settings/personal');
|
||||||
|
}
|
||||||
|
}
|
|
@ -52,12 +52,22 @@ class BackupCodeMapper extends Mapper {
|
||||||
}, $rows);
|
}, $rows);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param IUser $user
|
||||||
|
*/
|
||||||
public function deleteCodes(IUser $user) {
|
public function deleteCodes(IUser $user) {
|
||||||
|
$this->deleteCodesByUserId($user->getUID());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @param string $uid
|
||||||
|
*/
|
||||||
|
public function deleteCodesByUserId($uid) {
|
||||||
/* @var IQueryBuilder $qb */
|
/* @var IQueryBuilder $qb */
|
||||||
$qb = $this->db->getQueryBuilder();
|
$qb = $this->db->getQueryBuilder();
|
||||||
|
|
||||||
$qb->delete('twofactor_backup_codes')
|
$qb->delete('twofactor_backup_codes')
|
||||||
->where($qb->expr()->eq('user_id', $qb->createNamedParameter($user->getUID())));
|
->where($qb->expr()->eq('user_id', $qb->createNamedParameter($uid)));
|
||||||
$qb->execute();
|
$qb->execute();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue