Content-Length: 272966 | pFad | http://github.com/getsentry/sentry-javascript/issues/15970

AE Performance Issues After Sentry Configuration Update · Issue #15970 · getsentry/sentry-javascript · GitHub
Skip to content

Performance Issues After Sentry Configuration Update #15970

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.

Sign up for GitHub

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

Jump to bottom
Open
3 tasks done
NadiaSorokova opened this issue Apr 3, 2025 · 3 comments
Open
3 tasks done

Performance Issues After Sentry Configuration Update #15970

NadiaSorokova opened this issue Apr 3, 2025 · 3 comments
Assignees
@Christinarlong
Labels
Bug Next.js

Comments

@NadiaSorokova
Copy link

Is there an existing issue for this?

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/nextjs

SDK Version

8.47.0

Framework Version

No response

Link to Sentry event

No response

Reproduction Example/SDK Setup

No response

Steps to Reproduce

Added the following configuration to Sentry, and it's being fixed now after removing includeLocalVariables: true:

sentry.client.config:

import { feedbackIntegration, init, replayIntegration, setTags } from '@sentry/nextjs';

init({
  dsn: process.env.NEXT_PUBLIC_SENTRY_DSN || '',
  integrations: [
    replayIntegration(),
    ...(process.env.NEXT_PUBLIC_SENTRY_ENVIRONMENT === 'staging'
      ? [feedbackIntegration()]
      : [])
  ],
  tracesSampleRate: parseFloat(process.env.NEXT_PUBLIC_SENTRY_SAMPLE_RATE ?? '1'),
  replaysSessionSampleRate: 0.1,
  replaysOnErrorSampleRate: 1.0,
  debug: process.env.NEXT_PUBLIC_SENTRY_DEBUG === 'true' || false,

  includeLocalVariables: true,
  environment: process.env.NEXT_PUBLIC_SENTRY_ENVIRONMENT || '',
  allowUrls: [
    /https?:\/\/([a-z]+\.)?carhartt-wip\.(local|com(\/en-[A-Z]{2})?)/,
  ],
});

setTags({
  "vercel_link": process.env.NEXT_PUBLIC_VERCEL_URL,
  "device": window.innerWidth > 768 ? "desktop" : "mobile"
})

sentry.edge.config:

import { init } from '@sentry/nextjs';

init({
  dsn: process.env.NEXT_PUBLIC_SENTRY_DSN || '',
  tracesSampleRate: parseFloat(process.env.NEXT_PUBLIC_SENTRY_SAMPLE_RATE ?? '1'),
  debug: process.env.NEXT_PUBLIC_SENTRY_DEBUG === 'true' || false,
  includeLocalVariables: true,
  environment: process.env.NEXT_PUBLIC_SENTRY_ENVIRONMENT || '',
  allowUrls: [
    /https?:\/\/([a-z]+\.)?carhartt-wip\.(local|com(\/en-[A-Z]{2})?)/,
  ],
});

sentry.server.config:

import { init } from '@sentry/nextjs';

init({
  dsn: process.env.NEXT_PUBLIC_SENTRY_DSN || '',
  tracesSampleRate: parseFloat(process.env.NEXT_PUBLIC_SENTRY_SAMPLE_RATE ?? '1'),
  debug: process.env.NEXT_PUBLIC_SENTRY_DEBUG === 'true' || false,
  environment: process.env.NEXT_PUBLIC_SENTRY_ENVIRONMENT || '',
  includeLocalVariables: true,
  allowUrls: [
    /https?:\/\/([a-z]+\.)?carhartt-wip\.(local|com(\/en-[A-Z]{2})?)/,
  ],
});

Expected Result

The website’s performance should remain normal, without visible slowdowns or delays during page rendering

Actual Result

After adding the following config, the website's performance worsened. Pages started loading slowly. After removing the line includeLocalVariables: true, the performance returned to normal.
@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Apr 3, 2025
@github-actions github-actions bot added the Package: nextjs Issues related to the Sentry Nextjs SDK label Apr 3, 2025
@AbhiPrasad
Copy link
Member

@NadiaSorokova thanks for reporting.

Do you see similar performance slowdowns when running the project locally, or only in production environments?

@getsantry getsantry bot removed the status in GitHub Issues with 👀 3 Apr 3, 2025
@NadiaSorokova
Copy link
Author

@AbhiPrasad Hi 😊 It was slowed down both locally and in production environments

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Apr 7, 2025
@AbhiPrasad
Copy link
Member

Under the hood the local variables feature uses the nodejs inspector, which does have performance impact, but it shouldn't impact rendering performance like this for nextjs.

How complex is your next.js backend? Do you have an extensive react server components setup?

For now you'll have to disable this feature, but we'll take a look further to see what is going on.

@getsantry getsantry bot removed the status in GitHub Issues with 👀 3 Apr 8, 2025
@stephanie-anderson stephanie-anderson removed the Package: nextjs Issues related to the Sentry Nextjs SDK label Apr 16, 2025
@Christinarlong Christinarlong self-assigned this Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Christinarlong Christinarlong

Labels
Bug Next.js
Projects
GitHub Issues with 👀 3
Status: No status
Milestone
No milestone
Development

No branches or pull requests
4 participants
@stephanie-anderson @AbhiPrasad @Christinarlong @NadiaSorokova








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/getsentry/sentry-javascript/issues/15970

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy