Skip to content

Conversation

@cosmastech
Copy link
Contributor

@cosmastech cosmastech commented Nov 21, 2025

This will allow for reporting via report($requestException) to do its thing.

@taylorotwell
Copy link
Member

@cosmastech it seems like sometimes it returns null / void?

@cosmastech
Copy link
Contributor Author

@cosmastech it seems like sometimes it returns null / void?

If I remember correctly, it only carries forward in the reporting process IF it explicitly returns false.

https://github.com/laravel/framework/blob/12.x/src/Illuminate/Foundation/Exceptions/Handler.php#L380-L383

@cosmastech
Copy link
Contributor Author

Added a test to confirm the behavior.

@taylorotwell taylorotwell merged commit 240036c into laravel:12.x Nov 21, 2025
73 of 74 checks passed
@cosmastech cosmastech deleted the patch-29 branch November 23, 2025 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants