pada57 / serilog-sinks-influxdb

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

Add option to include/exclude any default fields #36 #37

Closed peitschie closed 1 year ago

peitschie commented 1 year ago

Addresses #36

peitschie commented 1 year ago

Personally, my preference is https://github.com/pada57/serilog-sinks-influxdb/pull/38 as a nice way to turn off all the default fields.

This one is a bit too fiddly in my opinion, and I don't like the need to either change the behaviour of IncludeSeverity or add an additional IncludeSeverityField (it's also likely missing a IncludeMessage option as well).

If you prefer this kind of approach though, I'm happy to extend it with the above tweaks to make not change any existing in-field behaviours and be purely opt-in.

peitschie commented 1 year ago

Actually, the more I fiddle with this, the less I like it. Closing this out in preference to my second PR.