-
-
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
Next. js version 12.3.4, invalid server report, no errors collected;"@sentry/nextjs": "8.13.0" #12837
Comments
Assigning to @getsentry/support for routing ⏲️ |
Can you confirm that this is only happening with Nextjs and this is not happening on any other language/platforms? |
@aldy505 I'm certainly not sure, but there's this situation next,However, what I have learned is that the next version on my end is 12.xx, which is a version issue |
I don't think this has anything to do with self-hosted. Since Sentry should receive all the data, I guess this is much likely a problem on the SDK itself. @hubertdeng123 @azaslavsky can we transfer this to sentry-javascript? |
@chargome Haha, I didn't downgrade,But I know the problem now, it was caused by next 12.xx |
@chargome This is a repetitive question I asked before, and I know how to solve it , Thank you;We are about to upgrade to 14.xx |
Good to hear you were able to resolve it! Upgrading to Next.js v14 is definitely a good idea to ensure ongoing compatibility with Sentry's latest features. Closing this, feel free to ping me if you need more help. |
Hi, I've downgraded to version 7.18, why do I need a second parameter for this? |
@chargome |
When I use withSentryGetServerSideProps, it seems to have become obsolete |
Hi @parlay96, not sure if I get your question. |
@chargome I know, but I would like to know why the second parameter of 'wrappGeserverSidePropsWithSentry' is? |
@chargome I am very grateful to you. Every time I come to see you for help, thank you |
In the screenshot you posted you can see it: |
@
Why wasn't the document example written? I am a mandatory item, what is its purpose |
@chargome Thank you, I'm going to follow you now |
@parlay96 you're right it should be part of the docs. So you want to pass the the route that will be parameterised there, e.g. |
Thank you |
next 12.x sentry7.1.. wrapGetServerSidePropsWithSentry It seems invalid, no errors have been collected, |
@chargome How do I check this question? Do I still need to raise issues for NextJS? |
@parlay96 in your code you are catching the error, hence no error is being captured by sentry. You can call |
@chargome I know it is possible to report captureException proactively and it can be implemented. Then why do I need to use 'wrapGetServerSidePropsWithSentry'? Wasn't 'wrapGetServerSidePropsWithSentry' just helping me automatically collect errors |
@parlay96 the SDK will report unhandled exceptions. If you want to send specific errors then use |
Thank you, I think I know how to use Sentry now |
@chargome ha ha ha , Thank you very much for solving my problem. You are my idol |
Environment
self-hosted (https://develop.sentry.dev/self-hosted/)
Steps to Reproduce
1: Firstly, the register method on the server was not executed at all,
2: The Sentry platform also did not collect any errors regarding the server side
3:Only client errors are effective for the client
Expected Result
How to solve it?
Actual Result
1: Firstly, the register method on the server was not executed at all,
2: The Sentry platform also did not collect any errors regarding the server side
3:Only client errors are effective for the client
Product Area
Unknown
Link
No response
DSN
No response
Version
"@sentry/nextjs": "8.13.0"
The text was updated successfully, but these errors were encountered: