evalphobia / logrus_sentry

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

Fix not to delete fields #25

Closed evalphobia closed 7 years ago

evalphobia commented 7 years ago

Resolve: #20

Instead directly delete key from logrus.Data, use wrapper struct dataField and omitList to exclude it from extra data.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+24.4%) to 88.542% when pulling 6952590b379d015c95886ad87d7a345e0d524880 on issue/20 into 3723d2d7cc850b6b5c738de1df8463211fb400c2 on master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+24.4%) to 88.542% when pulling 6952590b379d015c95886ad87d7a345e0d524880 on issue/20 into 3723d2d7cc850b6b5c738de1df8463211fb400c2 on master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+24.4%) to 88.542% when pulling 6952590b379d015c95886ad87d7a345e0d524880 on issue/20 into 3723d2d7cc850b6b5c738de1df8463211fb400c2 on master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+24.4%) to 88.542% when pulling 9d588aa1db4f9ba05df290ac74ce2ef44c6ee0cb on issue/20 into 3723d2d7cc850b6b5c738de1df8463211fb400c2 on master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+24.4%) to 88.542% when pulling 9d588aa1db4f9ba05df290ac74ce2ef44c6ee0cb on issue/20 into 3723d2d7cc850b6b5c738de1df8463211fb400c2 on master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+24.4%) to 88.542% when pulling 9d588aa1db4f9ba05df290ac74ce2ef44c6ee0cb on issue/20 into 3723d2d7cc850b6b5c738de1df8463211fb400c2 on master.

evalphobia commented 7 years ago

Thanks! 🐿