Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add Support for Golang's logfmt #562

Closed
blaggacao opened this issue Apr 13, 2018 · 4 comments
Closed

Add Support for Golang's logfmt #562

blaggacao opened this issue Apr 13, 2018 · 4 comments

Comments

@blaggacao
Copy link

blaggacao commented Apr 13, 2018

Please consider adding a parser, similar to json, for https://godoc.org/github.com/kr/logfmt
which is the default way of logging in everything golang, if no TTY is attached.

About 40% of my infrastructure containers speak this format, this might be quite average in some containerized environments, where everything's golang now a days...

@blaggacao blaggacao changed the title Add Support for logfmt Add Support for Golang's logfmt Apr 13, 2018
@blaggacao
Copy link
Author

blaggacao commented Apr 13, 2018

@edsiper I'd put a bounty out of current budget of USD 150,- on it. Let me know if and where to reward...

@edsiper
Copy link
Member

edsiper commented Nov 2, 2018

@blaggacao please coordinate with @manuelluis, he just implemented this feature.

@blaggacao
Copy link
Author

blaggacao commented Nov 2, 2018

Great news. @manuelluis, could you contact me on [email protected] for tx details. You should expose your signature attached to GitHub in any way to authenticate it's you.

@bluebike
Copy link
Contributor

bluebike commented Nov 2, 2018

Just comment to current structure of parsers.

I'm implementing parser for MariaDB audit logging currently it's just a filter,
but I could make that parser also, becuase it can parse (some) csv-files.

Currently parsers are "hardcoded" in flb_parsers.c/h.
Maybe parsers should be refactored to plugins (or similar structure).
There should be some kind of init/config, execute, close methods for those.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants