App crashes with Sentry console logging integration in SDK 9.37.0

[AbhiPrasad]

@coby.eastwood said:

Hey team, came across a problem when setting up logs for javascript, on our latest SDK version 9.37.0. My app will crash when enabling Sentry.consoleLoggingIntegration({levels: [‘log’]}), logging with Sentry.logger.warn('test'), and then using console log in beforeSendLog. Seems it is getting called recursively and crashes in chrome but in firefox it will show:

{
  "level": "info",
  "message": "beforeSendLog {\"level\":\"info\",\"message\":\"beforeSendLog {\\\"level\\\":\\\"info\\\",\\\"message\\\":\\\"beforeSendLog {\\\\\\\"level\\\\\\\":\\\\\\\"info\\\\\\\",\\\\\\\"message\\\\\\\":\\\\\\\"beforeSendLog {\\\\\\\\\\\\\\\"level\\\\\\\\\\\\\\\":\\\\\\\\\\\\\\\"info\\\\\\\\\\\\\\\",\\\\\\\\\\\\\\\"message\\\\\\\\\\\\\\\":\\\\\\\\\\\\\\\"beforeSendLog \ … continues… \"sentry.sdk.version\":\"9.37.0\"}}",
  "severityNumber": 10,
  "attributes": {
    "sentry.origin": "auto.console.logging",
    "sentry.sdk.name": "sentry.javascript.react",
    "sentry.sdk.version": "9.37.0"
  }
}

Made a repro here.

[mydea]

can we avoid this? 🤔 I guess we could call beforeSendLog in consoleSandbox...?

[AbhiPrasad]

yup that's the fix, we need to use consoleSandbox like we do for beforeBreadcrumb

[github-actions]

A PR closing this issue has just been released 🚀

This issue was referenced by PR #16968, which was included in the 9.39.0 release.