search

gemnasium / logrus-graylog-hook

Graylog hook for logrus
MIT License
87 stars 62 forks source link

added AddIgnoreSuffix #30

Open arteev opened 6 years ago

arteev commented 6 years ago

Greetings! There is a suggestion to add a method that extends the ignored file suffixes. This is necessary if logrus is wrapped in some logging library for the correct file and line fields. What do you think about this?

devishot commented 6 years ago

I would be happy if your idea accepted

gravis commented 6 years ago

I really like the change, unfortunately, we don't have any test for this change. Do you think you can add some to ensure it's working as expected? Thanks!

arteev commented 6 years ago

Oh sure! I will add tests. Thanks!

tossp commented 5 years ago

Related: sirupsen/logrus#989