search

pada57 / serilog-sinks-influxdb

Serilog Sinks for InfluxDB
MIT License
10 stars 9 forks source link

Added option to include or exclude HostName as tag #34

Closed linosteenkamp closed 1 year ago

linosteenkamp commented 1 year ago

Unit tests included

linosteenkamp commented 1 year ago

I have refactored my code to better align with the current implementation by overloading OptionalTag with the ability to take a boolean value to decide on including or excluding a tag.

Please consider making both level and severity optional tags, they basically duplicate each other. If you agree, I can create another PR or add to this one.

tom-englert commented 1 year ago

Please consider making both level and severity optional tags, they basically duplicate each other. If you agree, I can create another PR or add to this one.

Better add a new one