Add 'dontReportUsing' to filter exceptions to be reported (#56361)

* Add 'dontReportUsing' to filter exceptions to be reported

* Fix Handler->dontReportUsing() method

* Support for multiple calls and multiple callbacks for dontReportUsing()

* Add test

* Make test if statement nicer

* formatting

---------

Co-authored-by: Taylor Otwell <[email protected]>

Author: pelmered

src/Illuminate/Foundation/Configuration/Exceptions.php

@@ -150,6 +150,19 @@ public function dontReport(array|string $class)
         return $this;
     }
 
+    /**
+     * Register a callback to determine if an exception should not be reported.
+     *
+     * @param  callable  $using
+     * @return $this
+     */
+    public function dontReportWhen(Closure $dontReportWhen)
+    {
+        $this->handler->dontReportWhen($dontReportWhen);
+
+        return $this;
+    }
+
     /**
      * Do not report duplicate exceptions.
      *

src/Illuminate/Foundation/Exceptions/Handler.php

@@ -72,6 +72,14 @@ class Handler implements ExceptionHandlerContract
      */
     protected $dontReport = [];
 
+
+    /**
+     * The callbacks that inspect exceptions to determine if they should be reported.
+     *
+     * @var array
+     */
+    protected $dontReportCallbacks = [];
+
     /**
      * The callbacks that should be used during reporting.
      *
@@ -279,6 +287,23 @@ public function dontReport(array|string $exceptions)
         return $this->ignore($exceptions);
     }
 
+    /**
+     * Register a callback to determine if an exception should not be reported.
+     *
+     * @param  callable  $dontReportWhen
+     * @return $this
+     */
+    public function dontReportWhen(callable $dontReportWhen)
+    {
+        if (! $dontReportWhen instanceof Closure) {
+            $dontReportWhen = Closure::fromCallable($dontReportWhen);
+        }
+
+        $this->dontReportCallbacks[] = $dontReportWhen;
+
+        return $this;
+    }
+
     /**
      * Indicate that the given exception type should not be reported.
      *
@@ -413,6 +438,12 @@ protected function shouldntReport(Throwable $e)
             return true;
         }
 
+        foreach ($this->dontReportCallbacks as $dontReportCallback) {
+            if ($dontReportCallback($e) === true) {
+                return true;
+            }
+        }
+
         return rescue(fn () => with($this->throttle($e), function ($throttle) use ($e) {
             if ($throttle instanceof Unlimited || $throttle === null) {
                 return false;

tests/Foundation/FoundationExceptionsHandlerTest.php

@@ -635,6 +635,29 @@ public function testItCanDedupeExceptions()
         $this->assertSame($reported, [$one, $two]);
     }
 
+    public function testItCanSkipExceptionReportingUsingCallback()
+    {
+        $reported = [];
+        $e1 = new RuntimeException('foo');
+        $e2 = new RuntimeException('bar');
+
+        $this->handler->reportable(function (\Throwable $e) use (&$reported) {
+            $reported[] = $e;
+
+            return false;
+        });
+
+        $this->handler->dontReportWhen(function (\Throwable $e) {
+            return $e->getMessage() === 'foo';
+        });
+
+        $this->handler->report($e1);
+        $this->handler->report($e2);
+        $this->handler->report($e1);
+
+        $this->assertSame($reported, [$e2]);
+    }
+
     public function testItDoesNotThrottleExceptionsByDefault()
     {
         $reported = [];