diff options
| author |  Alexandre Alapetite <alexandre@alapetite.fr>
| 2025-04-01 17:55:20 +0200 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2025-04-01 17:55:20 +0200 |
| commit | 3336631a846c2e34f55b2ecb76b46cd68228486f (patch) | |
| tree | c2672609f84d57c1bf7c3c338a1369ff5748bff0 /app/Controllers/extensionController.php | |
| parent | dbdadbb4107878d9233f635c31a88afe45957101 (diff) | |
Catch extension exceptions in override (#7475)
* Catch extension exceptions in override
https://github.com/FreshRSS/Extensions/pull/300#issuecomment-2768578464
* Fix error message
Diffstat (limited to 'app/Controllers/extensionController.php')
| -rw-r--r-- | app/Controllers/extensionController.php | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/app/Controllers/extensionController.php b/app/Controllers/extensionController.php index eb2c3f8d8..f08013e7d 100644 --- a/app/Controllers/extensionController.php +++ b/app/Controllers/extensionController.php @@ -131,7 +131,12 @@ class FreshRSS_extension_Controller extends FreshRSS_ActionController { FreshRSS_View::prependTitle($ext->getName() . ' · ' . _t('admin.extensions.title') . ' · '); $this->view->extension = $ext; - $this->view->extension->handleConfigureAction(); + try { + $this->view->extension->handleConfigureAction(); + } catch (Minz_Exception $e) { // @phpstan-ignore catch.neverThrown (Thrown by extensions) + Minz_Log::error('Error while configuring extension ' . $ext->getName() . ': ' . $e->getMessage()); + Minz_Request::bad(_t('feedback.extensions.enable.ko', $ext_name, _url('index', 'logs')), ['c' => 'extension', 'a' => 'index']); + } } /** |