search

serilog-contrib / serilog-sinks-splunk

A Serilog sink that writes to Splunk
https://splunk.com
Apache License 2.0
46 stars 47 forks source link

Make it possible to override the text formatter implementation #43

Closed diegofrata closed 7 years ago

diegofrata commented 7 years ago

I needed to replace the SplunkJsonFormatter with my own implementation, this new constructor makes it easy to do so. Would be happy if this could make to the Nuget package! :)

merbla commented 7 years ago

@diegofrata LGTM, did you need an overload in the static extensions class?

diegofrata commented 7 years ago

I actually didn't since I'm deriving a class from it. But it looks like a good idea to add an overload there! I will resubmit the PR

diegofrata commented 7 years ago

Done.