Skip to content

Commit

Permalink
Merge pull request #45760 from nextcloud/backport/45355/stable25
Browse files Browse the repository at this point in the history
[stable25] fix: delete user credentials stored in storages_credentials when user gets deleted
  • Loading branch information
blizzz committed Jun 18, 2024
2 parents 858512b + daca6f8 commit 2eaa4f9
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions lib/private/User/Listeners/BeforeUserDeletedListener.php
Original file line number Diff line number Diff line change
Expand Up @@ -25,20 +25,23 @@

use OCP\EventDispatcher\Event;
use OCP\EventDispatcher\IEventListener;
use OCP\User\Events\BeforeUserDeletedEvent;
use OCP\Files\NotFoundException;
use OCP\IAvatarManager;
use OCP\Security\ICredentialsManager;
use OCP\User\Events\BeforeUserDeletedEvent;
use Psr\Log\LoggerInterface;

/**
* @template-implements IEventListener<BeforeUserDeletedEvent>
*/
class BeforeUserDeletedListener implements IEventListener {
private IAvatarManager $avatarManager;
private ICredentialsManager $credentialsManager;
private LoggerInterface $logger;

public function __construct(LoggerInterface $logger, IAvatarManager $avatarManager) {
public function __construct(LoggerInterface $logger, IAvatarManager $avatarManager, ICredentialsManager $credentialsManager) {
$this->avatarManager = $avatarManager;
$this->credentialsManager = $credentialsManager;
$this->logger = $logger;
}

Expand All @@ -61,5 +64,7 @@ public function handle(Event $event): void {
'exception' => $e,
]);
}
// Delete storages credentials on user deletion
$this->credentialsManager->erase($user->getUID());
}
}

0 comments on commit 2eaa4f9

Please sign in to comment.