Repository navigation
Next.js 13 app dir API route handlers #7228
Description
Problem Statement
Next.js released app dir route handlers: https://beta.nextjs.org/docs/routing/route-handlers
We should auto-instrument them.
Solution Brainstorm
We can add wrapper templates that wrap these files.
Stuff to consider:
- Static vs. dynamic routes
- Connected Traces
- Dynamic Route Segments
- Redirects
- Streaming responses
Reacted by Aron Jones, hayashi, Spyl, Ryan Mitchell, Scott Talbot, dmudro, FadiAboMsalam and Phil Wolstenholme
Reacted by FadiAboMsalam
Metadata
Metadata
Assignees
Projects
- Status
Activity
tomelliot commentedon Jul 11, 2023
As a temporary fix, I'm looking at using wrapApiHandlerWithSentry. Is it possible to use this with dynamic routes?
rsolomon commentedon Jul 19, 2023
@tomelliot Have you had luck getting
wrapApiHandlerWithSentryto play nicely with app routes?tomelliot commentedon Jul 20, 2023
@rsolomon sorry no progress to report
FadiAboMsalam commentedon Jul 27, 2023
Any update on this 🤔
am using app directory with latest "@sentry/nextjs": "^7.60.1"
But still no luck
lforst commentedon Jul 31, 2023
It's simply not implemented yet. It's on our backlog.
AbhiPrasad commentedon Sep 5, 2023
Released with
v7.67.0of the Next.js SDK. Feedback greatly appreciated! https://changelog.getsentry.com/announcements/nextjs-sdk-now-supports-route-handlersdmudro commentedon Sep 24, 2023
thanks @AbhiPrasad, the new implementation helps a lot.
assuming there is no need to wrap route handlers with
wrapRouteHandlerWithSentryas it's all handled by the Sentry Next SDK automatically, correct?lforst commentedon Sep 25, 2023
@dmudro route handlers should be wrapped automatically as long as
autoInstrumentServerFunctionsis not set tofalse.cyrus-za commentedon Jun 27, 2024
@lforst this isnt forking for me. I get all of my normal routes in my sentry performance tracing, but none of my api routes. Am I doing something wrong? I am using instrumentation.ts from the v8 migration docs and got tracing enabled in my sentry.client.ts
AbhiPrasad commentedon Jun 27, 2024
@cyrus-za a lot has changed with Next.js itself and the SDK since we released support for this. Could you open a new GH issue and fill out the template? Thanks!
cyrus-za commentedon Jul 5, 2024
This is solved for me. Turns out I had instrumentation.ts in the root rather than my src directory
lforst commentedon Jul 8, 2024
For future lurkers: If you have problems and stumble onto this issue, please open a new issue with reproduction steps!