-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
feat(nextjs): Instrument cache handler #13124
base: develop
Are you sure you want to change the base?
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
….com:getsentry/sentry-javascript into timfish/feat/nextjs-cache-handler-instrument
….com:getsentry/sentry-javascript into timfish/feat/nextjs-cache-handler-instrument
Ok, so a bit of an update of where I've got to... I've got this PR working in a demo app via ...but in the e2e tests, I've opened an issue to include |
Closes #12888
The nextjs custom cache handler is loaded via dynamic import, therefore to wrap its exported class we use
import-in-the-middle
which is already a dependency of@sentry/node
.Since nextjs is CJS, you can use a sub-module path to instrument the default handler by adding it as a custom cache handler like this:
Needs tests!