Merge pull request #34712 from nextcloud/backport/34688/stable25

[stable25] Don't crash with outdated share provider on update with the web updater

Author: szaimen

lib/private/Share20/ProviderFactory.php

@@ -40,13 +40,15 @@
 use OCA\FederatedFileSharing\TokenHandler;
 use OCA\ShareByMail\Settings\SettingsManager;
 use OCA\ShareByMail\ShareByMailProvider;
+use OCA\Talk\Share\RoomShareProvider;
 use OCP\Defaults;
 use OCP\EventDispatcher\IEventDispatcher;
 use OCP\IServerContainer;
 use OCP\Share\IManager;
 use OCP\Share\IProviderFactory;
 use OCP\Share\IShare;
 use OCP\Share\IShareProvider;
+use Psr\Log\LoggerInterface;
 
 /**
  * Class ProviderFactory
@@ -257,8 +259,15 @@ protected function getRoomShareProvider() {
 			}
 
 			try {
-				$this->roomShareProvider = $this->serverContainer->query('\OCA\Talk\Share\RoomShareProvider');
-			} catch (\OCP\AppFramework\QueryException $e) {
+				/**
+				 * @psalm-suppress UndefinedClass
+				 */
+				$this->roomShareProvider = $this->serverContainer->get(RoomShareProvider::class);
+			} catch (\Throwable $e) {
+				$this->serverContainer->get(LoggerInterface::class)->error(
+					$e->getMessage(),
+					['exception' => $e]
+				);
 				return null;
 			}
 		}
@@ -288,9 +297,16 @@ public function getProvider($id) {
 		}
 
 		foreach ($this->registeredShareProviders as $shareProvider) {
-			/** @var IShareProvider $instance */
-			$instance = $this->serverContainer->get($shareProvider);
-			$this->shareProviders[$instance->identifier()] = $instance;
+			try {
+				/** @var IShareProvider $instance */
+				$instance = $this->serverContainer->get($shareProvider);
+				$this->shareProviders[$instance->identifier()] = $instance;
+			} catch (\Throwable $e) {
+				$this->serverContainer->get(LoggerInterface::class)->error(
+					$e->getMessage(),
+					['exception' => $e]
+				);
+			}
 		}
 
 		if (isset($this->shareProviders[$id])) {
@@ -351,8 +367,17 @@ public function getAllProviders() {
 		}
 
 		foreach ($this->registeredShareProviders as $shareProvider) {
-			/** @var IShareProvider $instance */
-			$instance = $this->serverContainer->get($shareProvider);
+			try {
+				/** @var IShareProvider $instance */
+				$instance = $this->serverContainer->get($shareProvider);
+			} catch (\Throwable $e) {
+				$this->serverContainer->get(LoggerInterface::class)->error(
+					$e->getMessage(),
+					['exception' => $e]
+				);
+				continue;
+			}
+
 			if (!isset($this->shareProviders[$instance->identifier()])) {
 				$this->shareProviders[$instance->identifier()] = $instance;
 			}