use loggerinterface

Signed-off-by: Maxence Lange <[email protected]>
nextcloud · Jan 26, 2023 · 819cecf · 819cecf
1 parent 9e1914c
commit 819cecf
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/apps/files_sharing/lib/Controller/ShareAPIController.php b/apps/files_sharing/lib/Controller/ShareAPIController.php
@@ -81,6 +81,9 @@
 use OCP\Share\IManager;
 use OCP\Share\IShare;
 use OCP\UserStatus\IManager as IUserStatusManager;
+use Psr\Container\ContainerExceptionInterface;
+use Psr\Container\NotFoundExceptionInterface;
+use Psr\Log\LoggerInterface;
 
 /**
  * Class Share20OCS
@@ -410,8 +413,7 @@ private function fixMissingDisplayName(array $shares, ?array $updatedDisplayName
 	 * @param bool $cacheOnly - do not reach LUS, get data from cache.
 	 *
 	 * @return array
-	 * @psalm-suppress DeprecatedMethod
-	 * @psalm-suppress UndefinedClass
+	 * @throws ContainerExceptionInterface
 	 */
 	private function retrieveFederatedDisplayName(array $userIds, bool $cacheOnly = false): array {
 		// check if gss is enabled and available
@@ -422,9 +424,15 @@ private function retrieveFederatedDisplayName(array $userIds, bool $cacheOnly =
 		}
 
 		try {
+			/**
+			 * @psalm-suppress UndefinedClass
+			 */
 			$slaveService = Server::get(\OCA\GlobalSiteSelector\Service\SlaveService::class);
-		} catch (Exception $e) {
-			\OC::$server->getLogger()->logException($e);
+		} catch (\Throwable $e) {
+			Server::get(LoggerInterface::class)->error(
+				$e->getMessage(),
+				['exception' => $e]
+			);
 			return [];
 		}