-
Notifications
You must be signed in to change notification settings - Fork 247
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
don't rescue/log process exit or failure exceptions: SystemExit, SystemStackError, NoMemoryError, SecurityError, SignalException #533
Comments
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 20, 2021
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 20, 2021
This was referenced Mar 20, 2021
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 20, 2021
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 21, 2021
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 21, 2021
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 21, 2021
…to raise_expired_test_double_error
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 21, 2021
…to raise_expired_test_double_error
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 21, 2021
…to raise_expired_test_double_error
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 21, 2021
…to raise_expired_test_double_error
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 21, 2021
…to raise_expired_test_double_error
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 21, 2021
…to raise_expired_test_double_error
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 21, 2021
…to raise_expired_test_double_error
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 21, 2021
…to raise_expired_test_double_error
ColinDKelley
added a commit
to Invoca/listen
that referenced
this issue
Mar 22, 2021
Fixed by #535 in v3.5.0 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
listen
currently rescuesException
but this includes exceptions that indicate process exit or failure. For example, as reported in #532 by @RobinDaugherty:To address this,
listen
should simply rescueStandardError
(the default forrescue
).The text was updated successfully, but these errors were encountered: