search

evalphobia / logrus_sentry

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

Fix grouping and use a useful culprit with new SwitchExceptionTypeAndMessage option #58

Closed JelteF closed 6 years ago

JelteF commented 6 years ago

After getting some more errors I found out that the option I added didn't group errors if the message was not exactly the same. This fixes that by sending the stacktrace interface instead of the exception interface.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.09%) to 83.019% when pulling 89b4f9952372461797fe255f138d22cc53f456e4 on GetStream:fix-grouping-with-new-option into 61ba1582747fb74e7aff5fd397b66bb15ea6d103 on evalphobia:master.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.09%) to 83.019% when pulling 89b4f9952372461797fe255f138d22cc53f456e4 on GetStream:fix-grouping-with-new-option into 61ba1582747fb74e7aff5fd397b66bb15ea6d103 on evalphobia:master.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.09%) to 83.019% when pulling 89b4f9952372461797fe255f138d22cc53f456e4 on GetStream:fix-grouping-with-new-option into 61ba1582747fb74e7aff5fd397b66bb15ea6d103 on evalphobia:master.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.09%) to 83.019% when pulling 89b4f9952372461797fe255f138d22cc53f456e4 on GetStream:fix-grouping-with-new-option into 61ba1582747fb74e7aff5fd397b66bb15ea6d103 on evalphobia:master.

evalphobia commented 6 years ago

Thanks!

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