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

Start of sentry integration #1185

Merged
merged 11 commits into from
Jun 10, 2024
Merged

Start of sentry integration #1185

merged 11 commits into from
Jun 10, 2024

Conversation

TonyGiorgio
Copy link
Collaborator

@TonyGiorgio TonyGiorgio commented Jun 5, 2024

  • TODO option to turn it on/off
  • Parse mutiny-node console and decide when to submit errors
  • Find a better way to reference users/sessions than the default just anon for now, removed the default
  • Source maps / github integration
  • Delete all testing data
  • Make sure any invoice/txid data is scrubbed client side

Copy link

cloudflare-workers-and-pages bot commented Jun 5, 2024

Deploying mutiny-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5ef1f5b
Status: ✅  Deploy successful!
Preview URL: https://1d7c93b7.mutiny-web.pages.dev
Branch Preview URL: https://sentry.mutiny-web.pages.dev

View logs

@TonyGiorgio
Copy link
Collaborator Author

typescript checking is bogus, this is allowed: https://docs.sentry.io/platforms/javascript/configuration/filtering/#event-hints

@TonyGiorgio
Copy link
Collaborator Author

Best to test with the latest with mutiny-node in order to get better results.

@TonyGiorgio TonyGiorgio force-pushed the sentry branch 2 times, most recently from 8df7a15 to 0073fde Compare June 6, 2024 17:55
@TonyGiorgio
Copy link
Collaborator Author

TonyGiorgio commented Jun 6, 2024

Now requires turning on in the emergency toolkit

@TonyGiorgio TonyGiorgio force-pushed the sentry branch 2 times, most recently from 8f8ec94 to a7e092f Compare June 6, 2024 21:16
@TonyGiorgio TonyGiorgio marked this pull request as ready for review June 6, 2024 21:43
@TonyGiorgio TonyGiorgio requested a review from futurepaul June 6, 2024 21:44
Copy link
Collaborator

@futurepaul futurepaul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

working well for me, finding panics and errors!

@TonyGiorgio TonyGiorgio merged commit 4cab32d into master Jun 10, 2024
5 checks passed
@TonyGiorgio TonyGiorgio deleted the sentry branch June 10, 2024 15:43
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.

2 participants