diff options
| author | Joas Schilling <coding@schilljs.com> | 2021-05-05 10:41:18 +0200 |
|---|---|---|
| committer | Joas Schilling <coding@schilljs.com> | 2021-05-05 10:41:18 +0200 |
| commit | 784b059a0132e0e8685a9993bd74d570a5b75ec6 (patch) | |
| tree | 723727b4159b85eb65717b86f107ae7bf5a1f4ed /lib | |
| parent | 6d3aef1849eb84d423be2a8789bb13f6ef5a26a3 (diff) | |
| download | nextcloud-server-784b059a0132e0e8685a9993bd74d570a5b75ec6.tar.gz nextcloud-server-784b059a0132e0e8685a9993bd74d570a5b75ec6.zip | |
Don't break OCC if an app is breaking in it's Application class
Signed-off-by: Joas Schilling <coding@schilljs.com>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/private/AppFramework/Bootstrap/Coordinator.php | 29 | ||||
| -rw-r--r-- | lib/private/legacy/OC_App.php | 9 |
2 files changed, 24 insertions, 14 deletions
diff --git a/lib/private/AppFramework/Bootstrap/Coordinator.php b/lib/private/AppFramework/Bootstrap/Coordinator.php index 33b02c0291f..ad55ea3912e 100644 --- a/lib/private/AppFramework/Bootstrap/Coordinator.php +++ b/lib/private/AppFramework/Bootstrap/Coordinator.php @@ -113,21 +113,24 @@ class Coordinator { */ $appNameSpace = App::buildAppNamespace($appId); $applicationClassName = $appNameSpace . '\\AppInfo\\Application'; - if (class_exists($applicationClassName) && in_array(IBootstrap::class, class_implements($applicationClassName), true)) { - try { - /** @var IBootstrap|App $application */ - $apps[$appId] = $application = $this->serverContainer->query($applicationClassName); - } catch (QueryException $e) { - // Weird, but ok - continue; - } - try { + try { + if (class_exists($applicationClassName) && in_array(IBootstrap::class, class_implements($applicationClassName), true)) { + try { + /** @var IBootstrap|App $application */ + $apps[$appId] = $application = $this->serverContainer->query($applicationClassName); + } catch (QueryException $e) { + // Weird, but ok + continue; + } + $application->register($this->registrationContext->for($appId)); - } catch (Throwable $e) { - $this->logger->emergency('Error during app service registration: ' . $e->getMessage(), [ - 'exception' => $e, - ]); } + } catch (Throwable $e) { + $this->logger->emergency('Error during app service registration: ' . $e->getMessage(), [ + 'exception' => $e, + 'app' => $appId, + ]); + continue; } } diff --git a/lib/private/legacy/OC_App.php b/lib/private/legacy/OC_App.php index 65365c85e36..ed85253b649 100644 --- a/lib/private/legacy/OC_App.php +++ b/lib/private/legacy/OC_App.php @@ -135,7 +135,14 @@ class OC_App { ob_start(); foreach ($apps as $app) { if (!isset(self::$loadedApps[$app]) && ($types === [] || self::isType($app, $types))) { - self::loadApp($app); + try { + self::loadApp($app); + } catch (\Throwable $e) { + \OC::$server->get(LoggerInterface::class)->emergency('Error during app loading: ' . $e->getMessage(), [ + 'exception' => $e, + 'app' => $app, + ]); + } } } ob_end_clean(); |