App performance greatly reduced by `captureConsoleIntegration` #3992

yefim · 2024-08-05T15:43:07Z

OS:

Windows
MacOS
Linux

Platform:

iOS
Android

SDK:

@sentry/react-native (>= 1.0.0)
react-native-sentry (<= 0.43.2)

SDK version: 5.26.0

react-native version: 0.74.6

Are you using Expo?

Yes
No

Are you using sentry.io or on-premise?

sentry.io (SaaS)
on-premise

Configuration:

(@sentry/react-native)

  Sentry.init({
    dsn: process.env.EXPO_PUBLIC_SENTRY_DSN,
    enableInExpoDevelopment: false,
    enabled: !IS_DEV_BUILD,
    debug: IS_DEV_BUILD,
    environment: `app:${APP_ENV}`,
    integrations: [captureConsoleIntegration({ levels: ['error', 'assert'] })],
    ignoreErrors: [
      'Amplitude Logger [Error]',
      'ViewPropTypes will be removed from React Native',
      'POP_TO_TOP',
    ],
  });

I have the following issue:

After upgrading @sentry/integrations from "7.81.1" to "7.114.0" we noticed a large performance regression. The app was taking a long time to respond to a user's tap.

Before (`@sentry/integrations` 7.81.1)

RPReplay_Final1721856524.MP4

After (`@sentry/integrations` 7.114.0)

RPReplay_Final1721856599.MP4

When we enabled Sentry in dev, we saw a bunch of logs that were clogging up the console and were able to reproduce the unresponsiveness locally. I've uploaded a video at #3963 (comment)

Steps to reproduce:

Enable captureConsoleIntegration integration
Try navigating around the app

Actual result:

App is slow to respond to user's tap.

Expected result:

App navigates as normal

The text was updated successfully, but these errors were encountered:

kahest · 2024-08-06T12:14:34Z

@yefim thank you for the detailed report and the videos, we appreciate the effort. Do you know if this also occurs on Android, or only on iOS?

yefim · 2024-08-06T19:14:10Z

Yes, seeing the same against Android in development.

android-dev-slow.mp4

Here's the code diff:

krystofwoldrich · 2024-08-09T11:06:23Z

@yefim Thank you for the details, we will investigate it.

lucas-zimerman · 2024-08-16T00:17:50Z

It can be reproduced on the React Native sample by using the following init:

Sentry.init({
  // Replace the example DSN below with your own DSN:
  dsn: SENTRY_INTERNAL_DSN,
  debug: true,
  environment: 'dev',
  integrations(integrations) {
    integrations.push(
      captureConsoleIntegration({ levels: ['error', 'assert'] }),
    );
    return integrations;

  },
});

It seems like when we add the level 'assert' all this block appears, I noticed that there are loads of symbolication calls when this happens and navigation is surely almost impossible todo on the app. On the JavaScript debug log things seems calm but on logcat there is a nonstop calls to symbolicate

EDIT: I replaced the assert callback and this is the message + stacktrace from where it was being called.

 LOG  Assertion called: true  message 'this' is expected an Event object, but got [{"isTrusted":false}] stack : Error
    at getCurrentStackTrace (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:125938:26)
    at anonymous (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:125950:57)
    at apply (native)
    at anonymous (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:154465:27)
    at pd (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:36235:19)
    at setPassiveListener (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:36640:7)
    at dispatchEvent (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:36939:27)
    at setReadyState (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:36129:27)
    at open (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:36041:27)
    at apply (native)
    at anonymous (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:155583:34)
    at anonymous (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:29261:17)
    at tryCallTwo (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:204678:9)
    at doResolve (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:204817:25)
    at Promise (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:204697:14)
    at fetch (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:29218:25)
    at ?anon_0_ (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28649:33)
    at next (native)
    at asyncGeneratorStep (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28665:26)
    at _next (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28684:29)
    at anonymous (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28689:14)
    at tryCallTwo (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:204678:9)
    at doResolve (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:204817:25)
    at Promise (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:204697:14)
    at anonymous (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28681:25)
    at apply (native)
    at _symbolicateStackTrace (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28658:40)
    at apply (native)
    at symbolicateStackTrace (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28637:40)
    at symbolicate (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28616:52)
    at handleSymbolicate (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28516:42)
    at symbolicate (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28507:33)
    at appendNewLog (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28152:25)
    at anonymous (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:28196:21)
    at apply (native)
    at anonymous (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:35288:26)
    at _callTimer (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:35167:17)
    at _callReactNativeMicrotasksPass (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:35212:17)
    at callReactNativeMicrotasks (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:35418:44)
    at __callReactNativeMicrotasks (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:3569:48)
    at anonymous (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:3342:45)
    at __guard (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:3541:15)
    at flushedQueue (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:3341:21)
    at callFunctionReturnFlushedQueue (http://10.0.2.2:8081/index.bundle//&platform=android&dev=true&lazy=true&minify=false&app=io.sentry.reactnative.sample&modulesOnly=false&runModule=true:3326:33)

https://sentry-sdks.sentry.io/issues/5718158513

lucas-zimerman · 2025-03-18T00:12:13Z

I am stil looking at it, but in the meantime I have an update that this issue is no longer happening on @sentry/react-native version 6.0.0 or newer.
The fix was implemented here: https://github.com/getsentry/sentry-javascript/pull/11799/files#diff-54a4bb3b994992c0392eba31977780bac7892c266a8b0998ca213331a1eb57adR67

EDIT: It seems like the issue lies on this library used by React Native: https://gitlab.cin.ufpe.br/vrs2/iot-trafficlight-final/-/blob/main/node_modules/event-target-shim/dist/event-target-shim.mjs?ref_type=heads#L42-44

The symbolication code uses a fetch code that relies on this library, and with that it creates those assert messages, and since we are creating events with the fetched event, we will end up on a loop.

We ignore those events on the integration but it would be nice to shield it better against such issues in the future

yefim added the Platform: React-Native label Aug 5, 2024

getsantry bot added the Waiting for: Product Owner label Aug 5, 2024

github-project-automation bot added this to [DEPRECATED] Mobile SDKs Aug 5, 2024

github-project-automation bot moved this to Needs Discussion in [DEPRECATED] Mobile SDKs Aug 5, 2024

getsantry bot added this to GitHub Issues with 👀 3 Aug 5, 2024

getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Aug 5, 2024

yefim mentioned this issue Aug 5, 2024

Improve Session Replay performance on iOS #3959

Closed

11 tasks

getsantry bot removed the Waiting for: Product Owner label Aug 6, 2024

getsantry bot removed the status in GitHub Issues with 👀 3 Aug 6, 2024

getsantry bot added the Waiting for: Product Owner label Aug 6, 2024

getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Aug 6, 2024

kahest removed the Waiting for: Product Owner label Aug 6, 2024

getsantry bot removed the status in GitHub Issues with 👀 3 Aug 6, 2024

krystofwoldrich changed the title ~~App performance greatly reduced by captureConsoleIntegration on iOS~~ App performance greatly reduced by captureConsoleIntegration Aug 9, 2024

krystofwoldrich moved this from Needs Discussion to Needs Investigation in [DEPRECATED] Mobile SDKs Aug 9, 2024

krystofwoldrich assigned lucas-zimerman Aug 9, 2024

lucas-zimerman moved this from Needs Investigation to Backlog in [DEPRECATED] Mobile SDKs Aug 16, 2024

lucas-zimerman removed their assignment Aug 16, 2024

lucas-zimerman moved this from Backlog to Needs Discussion in [DEPRECATED] Mobile SDKs Aug 16, 2024

krystofwoldrich moved this from Needs Discussion to Backlog in [DEPRECATED] Mobile SDKs Aug 16, 2024

krystofwoldrich assigned lucas-zimerman Aug 16, 2024

krystofwoldrich unassigned lucas-zimerman Dec 10, 2024

krystofwoldrich assigned lucas-zimerman Mar 14, 2025

lucas-zimerman moved this from Backlog to In Progress in [DEPRECATED] Mobile SDKs Mar 17, 2025

sentry-triage-party bot added the getsentry/sentry-react-native label Apr 8, 2025

stephanie-anderson added Prio: high and removed getsentry/sentry-react-native labels Apr 9, 2025

lucas-zimerman linked a pull request Apr 22, 2025 that will close this issue

Fix: don't symbolicate console asserts that comes from metro. #4770

Draft

10 tasks

lucas-zimerman moved this from In Progress to Needs Review in [DEPRECATED] Mobile SDKs Apr 22, 2025

stephanie-anderson added Bug React-Native and removed Platform: React-Native labels May 7, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

App performance greatly reduced by `captureConsoleIntegration` #3992

App performance greatly reduced by `captureConsoleIntegration` #3992

yefim commented Aug 5, 2024

kahest commented Aug 6, 2024

Uh oh!

yefim commented Aug 6, 2024

Uh oh!

krystofwoldrich commented Aug 9, 2024

Uh oh!

lucas-zimerman commented Aug 16, 2024 •

edited

Loading

Uh oh!

lucas-zimerman commented Mar 18, 2025 •

edited

Loading

Uh oh!

Uh oh!

App performance greatly reduced by captureConsoleIntegration #3992

App performance greatly reduced by captureConsoleIntegration #3992

Comments

yefim commented Aug 5, 2024

Before (@sentry/integrations 7.81.1)

After (@sentry/integrations 7.114.0)

kahest commented Aug 6, 2024

Uh oh!

yefim commented Aug 6, 2024

Uh oh!

krystofwoldrich commented Aug 9, 2024

Uh oh!

lucas-zimerman commented Aug 16, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

lucas-zimerman commented Mar 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

App performance greatly reduced by `captureConsoleIntegration` #3992

App performance greatly reduced by `captureConsoleIntegration` #3992

Before (`@sentry/integrations` 7.81.1)

After (`@sentry/integrations` 7.114.0)

lucas-zimerman commented Aug 16, 2024 •

edited

Loading

lucas-zimerman commented Mar 18, 2025 •

edited

Loading