Skip to content

fix: Allow registering custom logger handlers - #1156

Merged
antusus merged 2 commits into
mainfrom
kb/allow_registering_custom_log_handlers
Mar 20, 2023
Merged

antusus merged 2 commits into
mainfrom
kb/allow_registering_custom_log_handlers

Conversation

@antusus

@antusus antusus commented Mar 16, 2023

Copy link
Copy Markdown
Contributor

We are being asked by users to allow configuring handlers. For example if you want to enable debug logging you will need to register new handler with allowed level set to at least FINE.

@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build #2827

  • 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.06%) to 71.222%

Totals Coverage Status
Change from base Build #2821: 0.06%
Covered Lines: 7108
Relevant Lines: 9980

💛 - Coveralls

@coveralls

coveralls commented Mar 16, 2023

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build #2838

  • 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.06%) to 71.222%

Totals Coverage Status
Change from base Build #2821: 0.06%
Covered Lines: 7108
Relevant Lines: 9980

💛 - Coveralls

@antusus
antusus merged commit 7373d5c into main Mar 20, 2023
@antusus
antusus deleted the kb/allow_registering_custom_log_handlers branch March 20, 2023 14:44
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

Successfully merging this pull request may close these issues.

3 participants