-
-
Notifications
You must be signed in to change notification settings - Fork 194
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unable to send events to GitLab's error-tracking backend #577
Comments
@Fnux hey, thanks for the issue! First question: did this work before 8.1.0? |
No, it didn't. I tried 1-2 weeks ago by checkout out the master branch (since the previous stable release did not work wit gitlab's DSN format), but it didn't work either. I waited for the 8.1 release before investigating further / opening this issue. |
Ah, perfect, okay. Follow up (possibly dumb) question... Did it ever work at any point? 😄 |
I never saw it working myself, but I believe it worked at some point since folks have explicitly patched sentry-elixir to support gitlab's DSN format: #506 |
I think this is an issue with Sentry envelopes, which were introduced with Sentry Relay. We use envelopes everywhere now, and these envelopes are encoded as multiple JSON objects separated by newlines. So, this is not gonna work unless GitLab updates their side of things. Supporting both envelopes as well as the old format is a big undertaking to do in this SDK, so I would recommend using a version prior to 8.1.0 if GitLab doesn't update their side of things. Thanks for reporting! 💟 |
Thanks for the update! |
Environment
I am using the recently released sentry-elixir
8.1.0
.Steps to Reproduce
mix sentry.send_test_event
.Expected Result
The event to be send / received on the gitlab side.
Actual Result
The endpoint is linked to a dummy gitlab project: https://gitlab.com/Fnux/sentry-demo
I would gladly provide any extra information and write write a patch if needed, but don't have the bandwidth to investigate right now.
The text was updated successfully, but these errors were encountered: