Support multiple values for severity level mapping

microsoft / fluent-plugin-application-insights

This is the fluentd output plugin for Azure Application Insights.

MIT License

15 stars 19 forks source link

Support multiple values for severity level mapping #5

Closed dennis-tra closed 5 years ago

dennis-tra commented 5 years ago

Hey there,

this is a non breaking change to specify multiple values of severity levels. I have the following problem: The Golang logging library Logrus has the log levels:

debug
info
warning
fatal
panic

fatal and panic should map to critical and it would be nice to just state:

  severity_level_critical fatal, panic

This PR addresses this problem.

Thanks for your time, Dennis

msftclas commented 5 years ago

All CLA requirements met.

yantang-msft commented 5 years ago

Thank you for the contribution! The PR looks pretty good, I'll merge and create a new gem soon after.