NextJS Sentry is using deprecated `beforeNavigate` instead of `beforeStartSpan`

[mpranjic24]

NextJS Sentry is using deprecated `beforeNavigate` instead of `beforeStartSpan`

[mpranjic24]

So we've just released v7.112.2, where we hopefully actually have a full fix for your problem - sorry about the confusion! Gonna leave this open this time until I've got confirmation that the issue is actually solved 😅

From what I can see, beforeStartSpan indeed gets called now 👍

Also, in the big "envelope/..." request I can see that the transaction property got changed in 2 places 👍

[mydea]

Hey, thanks for writing in! Hmm, this seems like a bug to me - we'll look into it!

[AbhiPrasad]

Fix released with https://github.com/getsentry/sentry-javascript/releases/tag/7.111.0 - please upgrade and give it a try!

[mpranjic24]

Unfortunately it doesn't seem to work, @AbhiPrasad. Nothing gets logged. On a side note, before the fix I could make it work with:

// @ts-expect-error: ignore TypeScript
      beforeNavigate(options: { name: string }) {
        ...
     }

That actually produced some console logs and in Chrome network view I could see that the transaction name got updated.

Now, it seems that it doesn't work at all.

[mydea]

I will look into it, it seems that something is really off there, sorry about that!

[mydea]

So we've just released v7.112.2, where we hopefully actually have a full fix for your problem - sorry about the confusion!
Gonna leave this open this time until I've got confirmation that the issue is actually solved 😅

[s1gr1d]

This seems to be fixed, so I am closing this 👍