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

Better support for compound loggers #25

Open
pyricau opened this issue Jan 2, 2024 · 0 comments
Open

Better support for compound loggers #25

pyricau opened this issue Jan 2, 2024 · 0 comments

Comments

@pyricau
Copy link
Member

pyricau commented Jan 2, 2024

Related to #22 and @ericmaxwell2003's feedback: while one can set up a custom logger that knows how to swap delegate loggers in and out, one challenge we found is that the API is split between "isLoggable" and "log". When delegating to multiple loggers, in the "log" call we have no idea which logger said yes to "isLoggable".

Not sure how to address this exactly, but the current API makes compound loggers awkward.

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

1 participant