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

Error while triggering instrumentation handler for INP #11156

Closed
3 tasks done
DurchblickerFabian opened this issue Mar 18, 2024 · 4 comments
Closed
3 tasks done

Error while triggering instrumentation handler for INP #11156

DurchblickerFabian opened this issue Mar 18, 2024 · 4 comments

Comments

@DurchblickerFabian
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

7.107.0

Framework Version

14.1.3

Link to Sentry event

No response

SDK Setup

Sentry.init({
  dsn: process.env.SENTRY_DSN,
  environment: process.env.SENTRY_ENV,
  autoSessionTracking: Boolean(process.env.NC_SENTRY_AUTO_SESSION_TRACKING),
  debug: process.env.NC_SENTRY_DEBUG === 'true',
  replaysSessionSampleRate: Number(process.env.NC_SENTRY_REPLAYS_SESSION_SAMPLE_RATE),
  replaysOnErrorSampleRate: Number(process.env.NC_SENTRY_REPLAYS_ON_ERROR_SAMPLE_RATE)
  integrations: [
    new Sentry.Replay(),
    Sentry.browserTracingIntegration({
      enableInp: true
    })
  ]
})

Steps to Reproduce

  1. Navigate from one page to another / also sometimes occurs when refreshing the page
  2. Error will occur

Expected Result

No error on navigation.

Actual Result

On page navigation the following error occurs:

image

sentry.log

@Lms24
Copy link
Member

Lms24 commented Mar 18, 2024

Hey @DurchblickerFabian thanks for writing in! I'm not entirely sure what's going wrong here but I traced it back to where I think the error is happening and opened a quick fix PR (#11157) to prevent the error.

I'm gonna tag @edwardgou-sentry and @AbhiPrasad to take a more detailed look because it's a bit unclear to me how we could run into the situation that the entry is missing.

Expect the fix to be released with the next version within this week. Thanks!

AbhiPrasad pushed a commit that referenced this issue Mar 19, 2024
…es not sending INP spans (#11183)

Fixes scenarios where interaction tracking wasn't properly recording up
to 10 candidate INP interactions.
Fixes `first-input` interaction not always tracked as INP candidates.
Fixes INP candidates not always using the longest latency even in their
interaction id grouping.

Also fixes: #11156
@jketcham
Copy link

I'm also experiencing this issue.

In my instance, the interactionIdtoRouteNameMapping argument to the _trackINP function was an empty object.

Debug screenshot Screenshot 2024-03-19 at 17 35 32

@Lms24
Copy link
Member

Lms24 commented Mar 20, 2024

Hi @jketcham this should be fixed via #11183 once it's released. Should happen in the next days.

@AbhiPrasad
Copy link
Member

Released with https://github.com/getsentry/sentry-javascript/releases/tag/7.108.0 - thanks for your patience!

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

No branches or pull requests

4 participants