Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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

Open
AGBrown opened this issue Aug 2, 2016 · 2 comments

Comments

@AGBrown
Copy link

AGBrown commented Aug 2, 2016

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
Copy link
Author

AGBrown commented Aug 2, 2016

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 5e2f37e on my machine anyway)

@tobiaszuercher
Copy link

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

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants