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

More logging options (v2.x) #38

Open
gaspaio opened this issue Aug 4, 2016 · 3 comments
Open

More logging options (v2.x) #38

gaspaio opened this issue Aug 4, 2016 · 3 comments

Comments

@gaspaio
Copy link

gaspaio commented Aug 4, 2016

At the moment we can only send logs to files. It would be nice (and necessary for my usecase) to be able to send them anywhere (syslog, logstash, ...).

Is this in the 2.0 roadmap somewhere ? Is this something that might be of interest ? if so, I can try to come up with a pull request.

I was thinking on using Logging. The memory logging it provides would probably allow for more extensive testing without filesystem dependency.

What d'ya think ?

@gaspaio gaspaio changed the title More logging options More logging options (v2.x) Aug 4, 2016
@ricbra
Copy link
Owner

ricbra commented Aug 4, 2016

Thanks for bringing this up. I've tried the suggested Logging library while writing v1, my argument against it at the time was that it didn't implement the default Go interface Writer. Perhaps I should reconsider, if the logging stays in at all.

I've been thinking about to completely remove the logging, and just outputting stuff to stdout and stderr. Leave the logging to something like supervisord which everybody is using for running as daemon? With supervisor you can send logs to files, syslog etc.

Or am I being rude and should I keep it? The less in this library, the less I have to maintain, the less bugs there will be.

@vicdelfant
Copy link

If I may put my two cents in: although a logging feature with a bunch of configuration options is great, I personally think it'd be best to keep this tool as lightweight and simple as possible. Logging to the right destination and acting upon it could/should be handled by a higher layer of the application stack instead.

We use supervisor (like you said, probably like everyone else) in a Docker container, so the lesser log files are created by default, the better :)

@gaspaio
Copy link
Author

gaspaio commented Sep 14, 2016

You're probably right, just stdout/stderr if probably enough. One can always redirect output with another tool. I'll let @ricbra close the issue if everybody has has its two cents :-). Thanks for the discussion.

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

No branches or pull requests

3 participants