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

Adjust logging in library packges #65

Open
SuperQ opened this issue Oct 24, 2021 · 2 comments
Open

Adjust logging in library packges #65

SuperQ opened this issue Oct 24, 2021 · 2 comments

Comments

@SuperQ
Copy link

SuperQ commented Oct 24, 2021

Using the chrony library forces users into using the logrus logging library. It's not a terrible choice, but it makes it such that you're stuck with it, even if you have other logging.

It would be nice if you could register a logger, similar to how we do this in gosnmp.

@abulimov
Copy link
Contributor

Thank you for reporting this problem. Indeed, makes sense to avoid making library users use particular logging framework. We'll check how it's done in gosnmp and come up with some solution.

@MovieStoreGuy
Copy link

This would be awesome to configure or even if it were to adopt the stdlib log package as part of the types.

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

4 participants
@SuperQ @abulimov @MovieStoreGuy and others