You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
raven-js 5.2/5.3 incompatible with sentry 4.9.8/5-dev #25
20:57 < joar> so, since raven-js is incompatible with sentry 5.0.0-dev, is 4.9.8 the way to go?
20:58 < joar> 4.9.8 is also incompatible with raven-js 5.3
21:00 < joar> oh, perhaps I should use raven-js 5.2 instead.
21:03 < joar> nope, CORS pre-flight request still fails.
21:05 -!- dz0ny [~dz0ny@84.41.89.147] has quit [Ping timeout: 276 seconds]
21:08 < joar> all right, using jQuery instead of zepto solves the CORS pre-flight request issue.
21:09 < joar> no wait, it doesn't.
21:09 < joar> but it at least tries to POST even though.
21:10 < joar> .. but when it posts, I receive an HTTP 400 reply, and the same issue as with sentry 5-dev and raven-js 5.3 comes into effect: https://gist.github.com/4603075e14a15fb1db77
21:20 < joar> same issue with raven-js 5.2 against app.getsentry.com.
21:22 < joar> oh wait, forgot to reload my config.
21:26 < joar> app.getsentry replies with an HTTP 400 and "Invalid data: Unable to validate interface, 'sentry.interfaces.Stacktrace':"
I tested it with Sentry 5.0.9, and I'm not having any problems. Perhaps there was an issue that was fixed between 5.0.0 and 5.0.9? In any case, can you upgrade and try again?
@unnali We're going to publicly release a beta this evening to get some other feedback and make sure it works for everyone before we actually tag a 1.0.
Activity
bkonkle commentedon Sep 10, 2012
Thanks! I'm going to start my day with raven-js and Sentry tomorrow, and I should be able to resolve this.
bkonkle commentedon Sep 11, 2012
I tested it with Sentry 5.0.9, and I'm not having any problems. Perhaps there was an issue that was fixed between 5.0.0 and 5.0.9? In any case, can you upgrade and try again?
joar commentedon Sep 11, 2012
Works perfectly from current master - that's awesome. Thank you for your help :)
joar commentedon Sep 11, 2012
Sadly I was wrong,
Raven.captureMessageworks, but notRaven.captureException(exc);. This usingraven-js 5.2and sentry master.peeter-tomberg commentedon Oct 9, 2012
The error is easily reproducible with RequireJS:
kivikakk commentedon Jan 16, 2013
I get 400s only with 0.7.1 and Sentry 5.1.4. :| Unsure if related.
mattrobenolt commentedon Jan 16, 2013
1.0 is currently in beta testing and will probably supersede all of these issues. :)
kivikakk commentedon Jan 16, 2013
I'm guessing this is the tracekit branch? I'll give it a go!
mattrobenolt commentedon Jan 16, 2013
@unnali Yep, and it requires Sentry 5.1.5, FYI. This has been completely rewritten to actually work.
kivikakk commentedon Jan 16, 2013
@mattrobenolt Thanks; have installed 5.1.5, will see if I can get them going through.
mattrobenolt commentedon Jan 16, 2013
@unnali Feel free to use: https://gist.github.com/raw/8921330d658b5f9aac94/raven-1.0-beta3.min.js
kivikakk commentedon Jan 16, 2013
holy crap, it worked! Well, you've got another beta tester. 🎆
kivikakk commentedon Jan 16, 2013
Okay, I'm feeling a lot better about this now. Thanks for the pointers and responsiveness! Looks like we may select Sentry after all. :3
mattrobenolt commentedon Jan 16, 2013
@unnali We're going to publicly release a beta this evening to get some other feedback and make sure it works for everyone before we actually tag a 1.0.
kivikakk commentedon Jan 16, 2013
@mattrobenolt Great!
Disable raven when NODE_ENV is something other than production
test(integration-tests): Add integration test scaffolding and release…