james-proxy / james

Web Debugging Proxy Application
1.42k stars 125 forks source link

Updates to use new Sentry browser/node clients #381

Closed mitchhentges closed 6 years ago

mitchhentges commented 6 years ago

The source mapping issue still exists, but our normalizeData function isn't as functional as it used to be

mitchhentges commented 6 years ago

Nevermind, looks like we've always been missing sourcemap data. I'll assign my :b:oi Tomi as a reviewer, since he set up Sentry the first time around (is there a reason I'm missing for why we should still manipulate the events Sentry sends?)