fix(#66): improve exception handling in ExceptionNotifyServiceProvider

- Add a check for method existence before calling reportable function in extendExceptionHandler method
guanguans · May 13, 2024 · b0b4f82 · b0b4f82
1 parent 8db8040
commit b0b4f82
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/ExceptionNotifyServiceProvider.php b/src/ExceptionNotifyServiceProvider.php
@@ -139,7 +139,10 @@ protected function registerTestCommand(): self
     protected function extendExceptionHandler(): self
     {
         $this->app->extend(ExceptionHandler::class, function (ExceptionHandler $exceptionHandler): ExceptionHandler {
-            if ($reportUsingCreator = config('exception-notify.report_using_creator')) {
+            if (
+                ($reportUsingCreator = config('exception-notify.report_using_creator'))
+                && method_exists($exceptionHandler, 'reportable')
+            ) {
                 /** @var callable(ExceptionHandler):callable|class-string $reportUsingCreator */
                 if (\is_string($reportUsingCreator) && class_exists($reportUsingCreator)) {
                     $reportUsingCreator = $this->app->make($reportUsingCreator);