Repository navigation
withServerActionInstrumentation catches NEXT_REDIRECT and NEXT_NOT_FOUND errors #10466
Description
Is there an existing issue for this?
- I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
- I have reviewed the documentation https://docs.sentry.io/
- I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases
How do you use Sentry?
Sentry Saas (sentry.io)
Which SDK are you using?
@sentry/nextjs
SDK Version
7.99.0
Framework Version
14.1.0
Link to Sentry event
SDK Setup
{
dsn: "https://ab637fbc48e826363f3cc7fbb1e0fc68@o4506114868117504.ingest.sentry.io/4506667113185280",
tracesSampleRate: 1,
debug: false,
}Steps to Reproduce
Create a server action with the following:
"use server";
import { withServerActionInstrumentation } from "@sentry/nextjs";
import { headers } from "next/headers";
export async function someAction() {
return await withServerActionInstrumentation(
"foo",
{ headers: headers() },
async () => {
redirect("/some-path");
},
);
}Expected Result
These exceptions should not be caught as per #7641.
Actual Result
An error is logged in Sentry issues:
Activity
lforst commentedon Feb 2, 2024
Ah right. Thanks for raising. This is an oversight but should be fixed easily. Backlogging!
fix(nextjs): Do not report redirects and notFound calls as errors in …
ishaangandhi commentedon Apr 4, 2024
What version of sentry do you have to have to no longer have this issue? I saw you completed this in #10474, but I am still running into this. I have
7.109.0.lforst commentedon Apr 5, 2024
@ishaangandhi would you mind creating a new issue with a reproduction example? Thank you!
MisterJimson commentedon May 17, 2024
I'm also seeing this still on newest v7 (7.116.0).
Show's up in Spotlight as well during local dev. Working on a reprod but not reproducible in an isolated way yet.