Symfony\Component\EventDispatcher: wrap calling the handlers into a try-catch block

Author: rotdrop

symfony/event-dispatcher/EventDispatcher.php

@@ -11,6 +11,9 @@
 
 namespace Symfony\Component\EventDispatcher;
 
+use Throwable;
+use Exception;
+
 use Psr\EventDispatcher\StoppableEventInterface;
 use Symfony\Component\EventDispatcher\Debug\WrappedListener;
 
@@ -217,7 +220,17 @@ protected function callListeners(iterable $listeners, string $eventName, object
             if ($stoppable && $event->isPropagationStopped()) {
                 break;
             }
-            $listener($event, $eventName, $this);
+            try {
+                $listener($event, $eventName, $this);
+            } catch (Throwable $t) {
+                // it is just not acceptable in such a heterogeneous infra
+                // structure like NC that one badly designed listener stops all
+                // others.
+                \OCP\Server::get(\Psr\Log\LoggerInterface::class)->error(
+                    'Exception during event dispatch',
+                    [ 'exception' => new \Exception('Exception during event dispatch', 0, $t) ],
+                );
+            }
         }
     }