Skip to content
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: add sentry #47

Closed
wants to merge 4 commits into from
Closed

feat: add sentry #47

wants to merge 4 commits into from

Conversation

scefali
Copy link

@scefali scefali commented Jun 27, 2023

This PR adds Sentry error, performance, and session replay monitoring. Note Sentry offers both a SaaS solution and
self-hosted solution. The instructions you are using SaaS but the guide still works with self-hosted with a few modifications.

Also note that because Next.js requires environment variables during build time for Sentry, we have to pass the environment variables through Docker arguments and modifying the local fly.toml which is in the .gitgnore (otherwise we wouldn't be able to add the sensitive environment variable to it).

Closes #28

@scefali
Copy link
Author

scefali commented Jul 6, 2023

@ykhli think you can have someone take a look?

@ykhli
Copy link
Collaborator

ykhli commented Jul 7, 2023

thank you for the PR @scefali ! Sorry for the delay - i have been traveling this week. Will take a look soon and let you know if there are questions.

Appreciate it! Excited to try this out

@timqian
Copy link
Collaborator

timqian commented Jul 7, 2023

Hi, @scefali Thanks for your contribution!
Sentry is a good plugin for people who need monitoring, but for people who want to start quickly, it may add extra complexity for them. So I'm a bit concerned about directly adding it to the template.
We are planning to build a CLI which allows users to choose which plugin they need. (#43) I think adding sentry in that way may be better for users

@scefali scefali closed this Aug 8, 2023
61cygni pushed a commit that referenced this pull request Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Application Monitoring
3 participants