Re-throw exception to invoke the native handler when a custom handler… #421
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This change causes captured exceptions to be propagated up to the native exception handler (re-thrown).
This behaviour is on by default and is controlled by a boolean variable passed to the
registerExceptionHandler
on theErrorHandler
.Right now exceptions are propagated to the existing exception handler if one exists (other than the native handler, meaning a custom function has been set using
set_exception_handler
).However if no custom exception handler is set exceptions will not propagate further than Sentry's handler. In other words exceptions are captured by Sentry but not re-thrown.
Because of this change in behaviour some may consider this a breaking change so we should consider the effects of it before release.
Because right now exceptions are not re-thrown the application is allowed to continue to execute and some applications may have unfortunately become dependent on this behaviour.
Those applications will face
Uncaught Exception
fatal errors if this change is activated for them.Related issue: #407