The on-by-default sendClientReports option breaks Web Workers

[mariyan-borisov]

• Review the documentation: https://docs.sentry.io/
• Search for existing issues: https://github.com/getsentry/sentry-javascript/issues
• Use the latest release: https://github.com/getsentry/sentry-javascript/releases
• Provide a link to the affected event from your Sentry account

Package + Version

• @sentry/browser
• @sentry/node
• raven-js
• raven-node (raven for node)
• other:

Version:

6.13.0

Description

My service worker registration fails because document is not available in service workers:

Uncaught ReferenceError: document is not defined

I consider this a breaking API change. This option should either be off by default, or it should not break Web Workers.

[AbhiPrasad]

Thanks for raising this issue. We have released a patch with the fix: https://www.npmjs.com/package/@sentry/browser/v/6.13.1. Please let us know if you encounter any other problems!

[mlahp7]

This fix is throwing some exceptions for us, and killing our workers. Any chance you can take a look at this fix?

#3994

[HazAT]

@mlahp7 we already released 6.13.1 which should fix this, can you try to upgrade?