Skip to content

raven-js 5.2/5.3 incompatible with sentry 4.9.8/5-dev #25

Closed

Description

@joar

Information:

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':"
-- https://gist.github.com/258a66f77781c143792b

Activity

  1. bkonkle commented on Sep 10, 2012

    @bkonkle
    Contributor

    Thanks! I'm going to start my day with raven-js and Sentry tomorrow, and I should be able to resolve this.

  2. bkonkle commented on Sep 11, 2012

    @bkonkle
    Contributor

    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?

  3. joar commented on Sep 11, 2012

    @joar
    Author

    Works perfectly from current master - that's awesome. Thank you for your help :)

  4. reopened this on Sep 11, 2012
  5. joar commented on Sep 11, 2012

    @joar
    Author

    Sadly I was wrong, Raven.captureMessage works, but not Raven.captureException(exc);. This using raven-js 5.2 and sentry master.

  6. peeter-tomberg commented on Oct 9, 2012

    @peeter-tomberg

    The error is easily reproducible with RequireJS:

    requirejs.onError = function(err) {
        Raven.captureException(err);
    }
    require("undefined");
  7. kivikakk commented on Jan 16, 2013

    @kivikakk

    I get 400s only with 0.7.1 and Sentry 5.1.4. :| Unsure if related.

  8. mattrobenolt commented on Jan 16, 2013

    @mattrobenolt
    Contributor

    1.0 is currently in beta testing and will probably supersede all of these issues. :)

  9. kivikakk commented on Jan 16, 2013

    @kivikakk

    I'm guessing this is the tracekit branch? I'll give it a go!

  10. mattrobenolt commented on Jan 16, 2013

    @mattrobenolt
    Contributor

    @unnali Yep, and it requires Sentry 5.1.5, FYI. This has been completely rewritten to actually work.

  11. kivikakk commented on Jan 16, 2013

    @kivikakk

    @mattrobenolt Thanks; have installed 5.1.5, will see if I can get them going through.

  12. mattrobenolt commented on Jan 16, 2013

    @mattrobenolt
    Contributor
  13. kivikakk commented on Jan 16, 2013

    @kivikakk

    holy crap, it worked! Well, you've got another beta tester. 🎆

  14. kivikakk commented on Jan 16, 2013

    @kivikakk

    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

  15. mattrobenolt commented on Jan 16, 2013

    @mattrobenolt
    Contributor

    @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.

  16. kivikakk commented on Jan 16, 2013

    @kivikakk
  17. added a commit that references this issue on Jun 12, 2018
    396531a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions