Skip to content

Commit a6133ab

Browse files
authored
Merge pull request #45355 from nextcloud/fix/delete-user-from-storage-credentials
fix: delete user credentials stored in storages_credentials when user gets deleted
2 parents 4391c73 + 7278198 commit a6133ab

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

lib/private/User/Listeners/BeforeUserDeletedListener.php

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
use OCP\EventDispatcher\IEventListener;
2828
use OCP\Files\NotFoundException;
2929
use OCP\IAvatarManager;
30+
use OCP\Security\ICredentialsManager;
3031
use OCP\User\Events\BeforeUserDeletedEvent;
3132
use Psr\Log\LoggerInterface;
3233

@@ -35,10 +36,12 @@
3536
*/
3637
class BeforeUserDeletedListener implements IEventListener {
3738
private IAvatarManager $avatarManager;
39+
private ICredentialsManager $credentialsManager;
3840
private LoggerInterface $logger;
3941

40-
public function __construct(LoggerInterface $logger, IAvatarManager $avatarManager) {
42+
public function __construct(LoggerInterface $logger, IAvatarManager $avatarManager, ICredentialsManager $credentialsManager) {
4143
$this->avatarManager = $avatarManager;
44+
$this->credentialsManager = $credentialsManager;
4245
$this->logger = $logger;
4346
}
4447

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

0 commit comments

Comments
 (0)