Logstash formatter #132
Logstash formatter #132
Conversation
|
i'm waiting for this to be merged already! 👍 |
|
The thing that makes me a little hesitant here is to have the formatter in the main directory. How do you feel about putting it in a |
|
@sirupsen I think it's good idea to have special directory for formatters. I will create such directory and update pull request. |
|
Thanks! |
|
I put formatter to special directory. |
| entry.Data["fields.type"] = v | ||
| } | ||
| entry.Data["type"] = f.Type | ||
| } |
There was a problem hiding this comment.
What about the msg field? Perhaps we should consider making prefixFieldClashes public so you can use it here?
There was a problem hiding this comment.
I replaced msg to message because Logstash uses message field for event description. So i can’t use prefixFieldClashes.
|
This needs tests, then it's ready to get in master. |
|
I will write tests during some days. |
|
@sirupsen Test is ready. |
|
@sirupsen Thanks. |
Hi,
Pull request contains Logstash Formatter which write output log as Logstash Event (http://logstash.net/).
My previous PR is #130
Thanks,
Alex Demidov