Sentry CDN ES 5 bundle is not ES 5 (includes class keyword)

[erikt9]

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 CDN bundle

SDK Version

7.44+

Framework Version

No response

Link to Sentry event

No response

SDK Setup

No response

Steps to Reproduce

The ES 5 bundles of 7.44.0 and 7.44.1 are not ES 5. They contain the class keyword. Previous versions do not.

View
https://browser.sentry-cdn.com/7.44.0/bundle.es5.min.js
https://browser.sentry-cdn.com/7.44.1/bundle.es5.min.js

/*! @sentry/browser 7.44.0 (82fdfad) | https://github.com/getsentry/sentry-javascript */
var Sentry=function(t){class n{static __initStatic(){ ...

Expected Result

ES 5 bundle should be ES 5

Actual Result

ES 5 bundle is ES 6

[mydea]

Hey @erikt9 ,

thanks for reporting. Not quite sure what is going on there, we'll investigate and ship a fix for this asap!

[mydea]

Hello,

this is resolved in 7.44.2! Sorry about the inconvenience. We'll add guards to make sure this doesn't happen anymore.

[kevireilly]

7.44.2 and 7.45.0 may still have an ES5 issue, though it may not be the exact same as reported here. We found that @sentry-internal/tracing needed to be explicitly transpiled.

Minified error

SyntaxError: Unexpected token '>'

Unminified error

SyntaxError: Unexpected token '('

Unminified source

function _autoloadDatabaseIntegrations() {
  const carrier = getMainCarrier();
  if (!carrier.__SENTRY__) {
    return;
  }
  const packageToIntegrationMapping = {
    mongodb() { // <<<<<<------ syntax error here
      const integration = dynamicRequire(module, './node/integrations/mongo');
      return new integration.Mongo();
    },
    mongoose() {
      const integration = dynamicRequire(module, './node/integrations/mongo');
      return new integration.Mongo({ mongoose: true });
    },
    mysql() {
      const integration = dynamicRequire(module, './node/integrations/mysql');
      return new integration.Mysql();
    },
    pg() {
      const integration = dynamicRequire(module, './node/integrations/postgres');
      return new integration.Postgres();
    },
  };
  const mappedPackages = Object.keys(packageToIntegrationMapping)
    .filter(moduleName => !!loadModule(moduleName))
    .map(pkg => {
      try {
        return packageToIntegrationMapping[pkg]();
      } catch (e) {
        return undefined;
      }
    })
    .filter(p => p) ;
  if (mappedPackages.length > 0) {
    carrier.__SENTRY__.integrations = [...(carrier.__SENTRY__.integrations || []), ...mappedPackages];
  }
}

[erikt9]

@kevireilly You'll probably want to make a new issue if you want the sentry team to look at it.

[mydea]

Yes, please create a new issue for this - esp. including: Are you talking about the ES5 CDN bundle? This is the only thing that we currently transpile. If you use the npm package, you have to take care of transpilation (and including polyfills) yourself, if you want to support ES5!