Gelf4NLog icon indicating copy to clipboard operation
Gelf4NLog copied to clipboard

Allow for additional fields to be set in the config file, (as per gelf4net)

Open AGBrown opened this issue 8 years ago • 2 comments

gelf4net (the log4net graylog appender) allows for additional fields to be set in the config file. It would be very useful if this dll did that too.

I will provide a PR as soon as I can.

AGBrown avatar Aug 02 '16 20:08 AGBrown

PR submitted, and it works fine. Note I wasn't able to work out how to get layout renderers to work with the config properties, e.g.

AdditionalFields="app:RandomSentence,version:1.0,Level:${level}"

This is mainly as I have no idea how those layout renderers work.

The ShouldSendLongUdpMessage test fails, but I don't think that is a result of what I've done (it was failing on 5e2f37e1 on my machine anyway)

AGBrown avatar Aug 02 '16 21:08 AGBrown

in my eyes a better approach: https://github.com/2020Legal/NLog.Targets.Gelf

tobiaszuercher avatar Sep 14 '16 09:09 tobiaszuercher