getsentry / sentry-javascript

Official Sentry SDKs for JavaScript
https://sentry.io
MIT License
7.95k stars 1.57k forks source link

Maximum call stack size exceeded #3026

Closed manny42 closed 3 years ago

manny42 commented 3 years ago

Package + Version

Version:

5.24.2

Description

The issue is Maximum call stack size exceeded the callstack is very short and only contains reference to code within the Sentry package.

This could be caused by the way we transform our code before shipping. We use browserify with babelify. Our babel config is:

module.exports = (api) => {
  const isTest = api.env('test');

  const plugins = [
    [
      '@babel/plugin-transform-classes',
      {
        loose: true,
      },
    ],
  ];

  const testTarget = {
    node: '12',
  };

  let defaultTarget = {
    chrome: '33',
    android: '36',
    ios: '9',
  };

  if (isTest) {
    defaultTarget = testTarget;
  }

  return {
    presets: [
      [
        '@babel/env',
        {
          targets: defaultTarget,
        },
      ],
    ],
    plugins,
  };
};

This started to happen when we upgraded from 4.6.6 to 5.24.2. That was the latest version when we made the upgrade. This issue was resolved as soon as we downgraded back to 4.6.6.

https://sentry.io/organizations/chartboost/issues/1956770806/?project=243530&query=is%3Aunresolved&sort=freq&statsPeriod=14d

rodolfoBee commented 3 years ago

The linked issue does not contain Sentry references in the stack trace and it is on v4.6.6.

manny42 commented 3 years ago

My bad I linked the ongoing issue when I should have linked a relevant event: https://sentry.io/organizations/chartboost/issues/1956770806/events/oldest/?project=243530&query=is%3Aunresolved&sort=freq&statsPeriod=14d

AbhiPrasad commented 3 years ago

Closing this issue for repo clean up. If you are still encountering this issue, please let us know. Thanks!