sentry-javascript/packages/utils/src/string.ts calls String.includes (breaks ES5 browsers) #7580

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? If you use the CDN bundles, please specify the exact bundle (e.g. bundle.tracing.min.js) in your SDK setup.

Sentry Browser Loader

SDK Version

7.44.2

Framework Version

No response

Link to Sentry event

No response

SDK Setup

No response

Steps to Reproduce

sentry-javascript/packages/utils/src/string.ts
Lines 106-108:

  if (isString(pattern)) {
    return requireExactStringMatch ? value === pattern : value.includes(pattern);
  }

value is defined to be a String.

String.includes is only defined in ES6, and will fail with ES5 only browsers (including IE 11). IE 11 and other ES5 browsers are listed as supported (as long as you are using the ES5 bundle) without polyfills (post Sentry version 5.7.0) in the matrix at https://docs.sentry.io/platforms/javascript/troubleshooting/supported-browsers/

Expected Result

Sentry to work in ES5 browsers without the need for polyfills, or documentation to be updated to state which polyfills are needed for the ES5 bundle to correctly work.

Actual Result

IE11 and other ES5 browsers throw exceptions when this code is invoked.