filestack / filestack-js

Official Javascript SDK for the Filestack API and content ingestion system.
https://www.filestack.com
MIT License
206 stars 76 forks source link

Improve Sentry exception handling #467

Open namoscato opened 2 years ago

namoscato commented 2 years ago

Expected Behavior

Ever since we fixed the Sentry integration, we've certainly been seeing exceptions flow through – they're just not very helpful.

Current Behavior

This is what most of them look like:

Screen Shot 2021-11-11 at 10 13 18 AM

Possible Solution

Here are a few ideas:

  1. Move to Sentry.captureMessage given that the current objects passed to captureException are clearly not Errors
  2. Remove the Sentry integration entirely (and just let users configure this in their application)

Steps to Reproduce (for bugs)

I can try to come up with an example Sentry event (without sensitive information) if necessary.

Additional Screenshots

Screen Shot 2021-11-11 at 10 17 45 AM

Context

Ultimately, we'd like to better understand / diagnose Filestack picker upload-related errors. This includes better fingerprinting in addition to exception context.

Your Environment