Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate to sentry v8 sdk #8

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Migrate to sentry v8 sdk #8

wants to merge 2 commits into from

Conversation

CptJJ
Copy link

@CptJJ CptJJ commented May 14, 2024

Three main issues, calling in @lforst

  1. Client side unhandled errors throw a Maximum call stack size exceded: Sentry in Vue app causes freeze / 100% CPU usage (inf. recursion) [includes minimal example] getsentry/sentry-javascript#2957 (comment)

  2. Console errors dont seem to trigger the sdk via: Sentry.captureConsoleIntegration({levels:['error']}),

  3. Nitro server seems to break due to the @sentry/node init() function. ive gotten a range of strange errors such as "vue does not export computed" and "no such file or directory, open '/.../node_modules/vue/server-renderer/@vue/server-renderer' Which makes me think maybe its an issue on my end?

@CptJJ CptJJ changed the title Being work on migration to sentry sdk v8 Migrate to sentry v8 sdk May 14, 2024
uncomment out all the integrations.
@CptJJ
Copy link
Author

CptJJ commented May 24, 2024

@lforst were you able to look into this at all? still having this issue in v8.3.0

@lforst
Copy link

lforst commented Jun 25, 2024

@CptJJ Don't know if you saw but we started to work on a dedicated nuxt 3 (and 4) SDK. getsentry/sentry-javascript#9095

@RayGuo-ergou
Copy link

Nitro server seems to break due to the @sentry/node init() function. ive gotten a range of strange errors such as "vue does not export computed" and "no such file or directory, open '/.../node_modules/vue/server-renderer/@vue/server-renderer' Which makes me think maybe its an issue on my end?

Found a workaround for this one

// sentry.ts
globalThis._sentryEsmLoaderHookRegistered = true;

From: getsentry/sentry-javascript#12807

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants