Closed Bug 965040 Opened 12 years ago Closed 10 years ago

Integrate RavenJS with Fireplace

Categories

(Marketplace Graveyard :: Code Quality, defect, P4)

Avenir
x86
macOS
defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: jlockhart, Unassigned)

Details

(Whiteboard: [marketplace-transition])

Integrate RavenJS to point to a Zamboni endpoint that forwards to Sentry over the VPN. Example of how this was achieved with Webpay is here: https://github.com/jaredkerim/webpay/commit/518080915e9d907c46595147795a05c7abb4a561
Component: General → Code Quality
Priority: -- → P4
Gosh darn! Raven.js is huge: http://cdn.ravenjs.com/1.1.16/raven.js The native and jQuery versions weigh in at 59 KB / 62 KB uncompressed (5.7 KB / 5.9 KB minified+gzip'd).
I tried following this: https://github.com/mozilla/webpay/commit/5180809 But I wasn't comfortable with including `raven.js` in Fireplace, considering how large it is (5.7 KB isn't huge, but still). I think we should look into it more to really see what it's all doing. Notice that Raven.js was added to webpay (bug 942912) but it appears to not actually have a URL set up: https://github.com/mozilla/webpay/blob/2d51adf/webpay/settings/base.py#L401-L407
PR: https://github.com/mozilla/fireplace/pull/730 I have opened this PR as a WIP alternative to Raven.js. I have immediately closed the PR, but anyone is free to resurrect. One day we should consider actually implementing Raven.js. It's irresponsible to not log client-side errors. We would never think for a second about getting rid of Sentry for our Django app, so we should definitely do the same logging with our front-end app.
I just found this today! http://requestb.in/ It's essentially what I wanted to build - but luckily someone else built it and it's way better. Super useful for testing.
We're enrolled in a trial of New Relic Browser™, which does JS errors (among other perf analytics stuff). And now we've got JS errors in Marketplace production rolling in: https://rpm.newrelic.com/accounts/315282/browser/2914756/js_errors And it's sweeeeet!
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Whiteboard: [marketplace-transition]
You need to log in before you can comment on or make changes to this bug.