Skip to content
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

Original Exception not included as part of the hint when triggering the beforeSend function #80240

Open
dcarabott opened this issue Nov 5, 2024 · 3 comments

Comments

@dcarabott
Copy link

dcarabott commented Nov 5, 2024

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

When capturing an exception manually, the eventHint.originalException is not being set before triggering the beforeSend function.

client.captureException(exception);

captureException signature is "captureException(exception, hint, scope)"

this will internally call

 this._captureEvent(event, hint, scope) -> this._processEvent(event, hint, scope)

and _processEvent calls the processBeforeSend without modifying hint:

processBeforeSend(options, prepared, hint)

Expected Result

Hint to include the original exception before triggering beforeSend

Actual Result

The beforeSend function has no access to the originalException.

Product Area

Unknown

Link

No response

DSN

No response

Version

JS SDK Version: 7.119.2

@getsantry
Copy link
Contributor

getsantry bot commented Nov 5, 2024

Assigning to @getsentry/support for routing ⏲️

@InterstellarStella
Copy link

Hi @dcarabott which SDK package and version are you using?

@dcarabott
Copy link
Author

Hi @InterstellarStella, we're using the JS SDK through the CDN link. Version right now is 7.119.2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Waiting for: Product Owner
Development

No branches or pull requests

2 participants