Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Use @sentry/svelte instead of @sentry/sveltekit
See getsentry/sentry-javascript#8291 Currently not supported in non-node environments
- Loading branch information
1 parent
7b160e6
commit a2b94bd
a2b94bdThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @helblingjoel ! I was looking for a solution to add Sentry to a Sveltekit project,
Did this modification worked for you ?
Thank you in advance & have a nice day !
a2b94bdThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @MarArMar
That depends on what you mean by "work" 😅
I previously switched my runtime to CF and deployments started to fail.
I still get Sentry on anything that's happening client side, but lose server-side observability. I'm fine with this as this is still under active development.
I'm really hoping that Sentry will have made the relevant changes by the time this project is closer to being production ready - not having BE observability is quite a big gap.
Hope this helps, and you too!
a2b94bdThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, I've not checked back on Sentry in a while, seems like something with the data flow has broken.
I'll try to fix it and tag you in the commit that fixes it
a2b94bdThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update: It could work, I had something misconfigured in Sentry that blocked all new incoming traffic...
I realised this after switching to @sentry/browser. Regardless, you might still find this interesting: 00ecbe1
a2b94bdThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed, interesting, many thanks for your answer !!
Have a nice week-end 🙂