feat: Catch callback exception in callForSeenUsersartonge/feat/allow_partial_seen_users

In case of large instances, `callForSeenUsers()` can take a lot of time to finish.

The new arguments give the opportunity for the caller to set a time limit, and to start from a given offset. The method now also returns the offset if it was stopped early.

Signed-off-by: Louis Chemineau <louis@chmn.me>

1 files changed, 7 insertions, 3 deletions

diff --git a/lib/private/User/Manager.php b/lib/private/User/Manager.php
index a30d66868e4..6904093d045 100644
--- a/lib/private/User/Manager.php
+++ b/lib/private/User/Manager.php
@@ -637,9 +637,13 @@ class Manager extends PublicEmitter implements IUserManager {
 				foreach ($this->backends as $backend) {
 					if ($backend->userExists($userId)) {
 						$user = $this->getUserObject($userId, $backend, false);
-						$return = $callback($user);
-						if ($return === false) {
-							return 0;
+						try {
+							$return = $callback($user);
+							if ($return === false) {
+								return 0;
+							}
+						} catch (\Throwable $e) {
+							$this->logger->error('Error while calling callback for seen users', ['exception' => $e, 'userId' => $userId, 'backend' => $backend::class]);
 						}
 						break;
 					}