-
Notifications
You must be signed in to change notification settings - Fork 10.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[10.x] Add job timeout occurred event #47068
[10.x] Add job timeout occurred event #47068
Conversation
Thanks for your contribution to Laravel! Would you be willing to include some tests in this pull request? |
To test when the timeout occurred, the process is killed, so the assertion cannot be checked. |
@saeedhosseiinii No problem. Testing something as simple as this could be complicated and not worth the hassle. Therefore, let's have Taylor reviewing the pull request without tests. |
You can register an exception handler (ExceptionHandler::reportable method) to handle the new |
This exception only occurs in the JobFailed event when the job fails. |
No event is dispatched when timeout occurss, so there is no way for external packages to know about it. With this pr, this is possible