SentryError constructor fail with a TypeError if Object/Error prototype has been frozen

[tomi]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/node

SDK Version

8.52.0

Framework Version

'@sentry/node': 8.52.0

Link to Sentry event

https://n8nio.sentry.io/issues/6252828797/events/4dcec130b6a94456a6693fd4d81ffc89/?project=4508342481780736

Reproduction Example/SDK Setup

No response

Steps to Reproduce

1. Freeze prototypes of global objects. Object.prototype and Error.prototype are probably enough

Object.freeze(Object.prototype)
Object.freeze(Error.prototype)

2. Initialize sentry and add a beforeSend function that returns null
3. Capture an error

Expected Result

No errors are thrown from Sentry code

Actual Result

This error is thrown and reported:

TypeError
Cannot assign to read only property 'name' of object 'Error: before send for type `error` returned `null`, will not send event.'

Please see the attached Sentry event link for more details

[mydea]

hey, thanks for writing in! I will look into this, it seems to be caused by what we do in the SentryError class, maybe we can get rid of some of that behavior.