Merge pull request #154 from stackkit/bugfix/report-errors

marickvantuil · web-flow · commit d0aeecd79af8 · 2024-08-26T22:00:01.000+02:00
Fix exceptions in jobs not reported to exception handler
diff --git a/src/TaskHandler.php b/src/TaskHandler.php
@@ -41,7 +41,7 @@ public function handle(?string $task = null): void
         // is returned. Because we manually manage retries by releaseing jobs,
         // we never want to return a 5xx status as that will result in duplicate
         // job attempts.
-        rescue(fn () => $this->run($task), report: false);
+        rescue(fn () => $this->run($task));
     }
 
     private function run(IncomingTask $task): void

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ public function handle(?string $task = null): void`
`41`	`41`	`// is returned. Because we manually manage retries by releaseing jobs,`
`42`	`42`	`// we never want to return a 5xx status as that will result in duplicate`
`43`	`43`	`// job attempts.`
`44`		`- rescue(fn () => $this->run($task), report: false);`
	`44`	`+ rescue(fn () => $this->run($task));`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`private function run(IncomingTask $task): void`