spacefreak/user_sql
1
0
Fork 0
Code Pull Requests Activity

Use new dispatcher for password policy event

Browse Source 
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
This commit is contained in:
Morris Jobke
2020-07-09 21:56:14 +02:00
parent 5ee566f9b5
commit 5facd20111
2 changed files with 12 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
appinfo/info.xml
View File

@@ -22,7 +22,7 @@
<category>auth</category> <category>auth</category>
<dependencies> <dependencies>
<php min-version="7.1"/> <php min-version="7.1"/>
<nextcloud min-version="14" max-version="19"/> <nextcloud min-version="18" max-version="19"/>
</dependencies> </dependencies>
<settings> <settings>
<admin>\OCA\UserSQL\Settings\Admin</admin> <admin>\OCA\UserSQL\Settings\Admin</admin>

16
lib/Backend/UserBackend.php
View File

@@ -34,9 +34,11 @@ use OCA\UserSQL\Crypto\IPasswordAlgorithm;
use OCA\UserSQL\Model\User; use OCA\UserSQL\Model\User;
use OCA\UserSQL\Properties; use OCA\UserSQL\Properties;
use OCA\UserSQL\Repository\UserRepository; use OCA\UserSQL\Repository\UserRepository;
use OCP\EventDispatcher\IEventDispatcher;
use OCP\IConfig; use OCP\IConfig;
use OCP\IL10N; use OCP\IL10N;
use OCP\ILogger; use OCP\ILogger;
use OCP\Security\Events\ValidatePasswordPolicyEvent;
use OCP\User\Backend\ABackend; use OCP\User\Backend\ABackend;
use OCP\User\Backend\ICheckPasswordBackend; use OCP\User\Backend\ICheckPasswordBackend;
use OCP\User\Backend\ICountUsersBackend; use OCP\User\Backend\ICountUsersBackend;
@@ -96,6 +98,10 @@ final class UserBackend extends ABackend implements
* @var EventDispatcher The event dispatcher. * @var EventDispatcher The event dispatcher.
*/ */
private $eventDispatcher; private $eventDispatcher;
/**
* @var IEventDispatcher The new event dispatcher.
*/
private $newEventDispatcher;
/** /**
* @var IUserAction[] The actions to execute. * @var IUserAction[] The actions to execute.
*/ */
@@ -112,11 +118,12 @@ final class UserBackend extends ABackend implements
* @param IL10N $localization The localization service. * @param IL10N $localization The localization service.
* @param IConfig $config The config instance. * @param IConfig $config The config instance.
* @param EventDispatcher $eventDispatcher The event dispatcher. * @param EventDispatcher $eventDispatcher The event dispatcher.
* @param IEventDispatcher $newEventDispatcher The new event dispatcher.
*/ */
public function __construct( public function __construct(
$AppName, Cache $cache, ILogger $logger, Properties $properties, $AppName, Cache $cache, ILogger $logger, Properties $properties,
UserRepository $userRepository, IL10N $localization, IConfig $config, UserRepository $userRepository, IL10N $localization, IConfig $config,
EventDispatcher $eventDispatcher EventDispatcher $eventDispatcher, IEventDispatcher $newEventDispatcher
) { ) {
$this->appName = $AppName; $this->appName = $AppName;
$this->cache = $cache; $this->cache = $cache;
@@ -126,6 +133,7 @@ final class UserBackend extends ABackend implements
$this->localization = $localization; $this->localization = $localization;
$this->config = $config; $this->config = $config;
$this->eventDispatcher = $eventDispatcher; $this->eventDispatcher = $eventDispatcher;
$this->newEventDispatcher = $newEventDispatcher;
$this->actions = []; $this->actions = [];
$this->initActions(); $this->initActions();
@@ -515,10 +523,8 @@ final class UserBackend extends ABackend implements
return false; return false;
} }
$event = new GenericEvent($password); $event = new ValidatePasswordPolicyEvent($password);
$this->eventDispatcher->dispatch( $this->newEventDispatcher->dispatchTyped($event);
'OCP\PasswordPolicy::validate', $event
);
$user = $this->userRepository->findByUid($uid); $user = $this->userRepository->findByUid($uid);
if (!($user instanceof User)) { if (!($user instanceof User)) {