search

sentry-extensions / cyclops

Cyclops is meant to be a high performance barrier in front of Sentry (http://getsentry.com).
http://sentry-extensions.github.io/cyclops/
164 stars 15 forks source link

Big refactoring of grouping logic that fixes bugs with new sentry clients #25

Closed WoZ closed 7 years ago

WoZ commented 7 years ago

Big refactoring of grouping logic. Cyclops checks stacktraces first like Sentry does and then uses message. Fingerprint isn't supported yet.

See tests test_grouping test for the new logic. Fully tested on raven-js 1.3+, sentry-php 0.12+ and raven-node 0.5+

ghost commented 7 years ago

Hi Dimitry, I'm coming back to you as you are the only one who is active in this project. I want to know if you experienced this issue before with cyclops: Cyclops get the errors without any problem but all the errors are IGNORED. have you already seen that before ?

WoZ commented 7 years ago

May you reproduce that? What sentry client do you use? Do you have a payload of events that were sent to cyclops?

8 февр. 2017 г. 15:33 пользователь "badr-badri" notifications@github.com написал:

Hi Dimitry, I'm coming back to you as you are the only one who is active in this project. I want to know if you experienced this issue before with cyclops: Cyclops get the errors without any problem but all the errors are IGNORED. have you already seen that before ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/heynemann/cyclops/pull/25#issuecomment-278329619, or mute the thread https://github.com/notifications/unsubscribe-auth/AAFPHhP32t9AqBqNap7w9m_pkUAcE5r3ks5racQbgaJpZM4LAHPW .