search

evalphobia / logrus_sentry

sentry hook for logrus
MIT License
194 stars 78 forks source link

Support fingerprinting #39

Closed DoHe closed 7 years ago

DoHe commented 7 years ago

Sentry uses an array of strings as a fingerprint. This allows to group messages that otherwise wouldn't be grouped.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.4%) to 83.754% when pulling d44fa35b1ad83bd0fcf75f0c83c9c53ffc2689bc on ecosia:master into 162fd93cca6f0b1170f1b12c980a004744297f13 on evalphobia:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.4%) to 83.754% when pulling d44fa35b1ad83bd0fcf75f0c83c9c53ffc2689bc on ecosia:master into 162fd93cca6f0b1170f1b12c980a004744297f13 on evalphobia:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.4%) to 83.754% when pulling d44fa35b1ad83bd0fcf75f0c83c9c53ffc2689bc on ecosia:master into 162fd93cca6f0b1170f1b12c980a004744297f13 on evalphobia:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.4%) to 83.754% when pulling d44fa35b1ad83bd0fcf75f0c83c9c53ffc2689bc on ecosia:master into 162fd93cca6f0b1170f1b12c980a004744297f13 on evalphobia:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.4%) to 83.754% when pulling d44fa35b1ad83bd0fcf75f0c83c9c53ffc2689bc on ecosia:master into 162fd93cca6f0b1170f1b12c980a004744297f13 on evalphobia:master.

evalphobia commented 7 years ago

@DoHe Thank you for contribution!

Would you add fingerprint to Special fields section on README?

DoHe commented 7 years ago

@evalphobia Sure thing, thanks for writing this in the first place :wink: I added it to the documentation. For now I'm referring to the official documentation as the logic is not that straight forward and I think they are doing a better job of explaining it than I could do in a few lines here. If you still prefer me to add a short description of the logic or not link to the documentation, let me know.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.1%) to 83.473% when pulling 4dd7a97d83fa840def203bd297a8427c5d9087b3 on ecosia:master into 162fd93cca6f0b1170f1b12c980a004744297f13 on evalphobia:master.

evalphobia commented 7 years ago

It's okay, thank you! ✌️

Note: https://github.com/evalphobia/logrus_sentry/releases/tag/v0.2.12