Skip to content

Fix excessive logging from druid-basic-security#16767

Merged
kfaraz merged 1 commit intoapache:masterfrom
dave-mccowan:fixlog
Jul 22, 2024
Merged

Fix excessive logging from druid-basic-security#16767
kfaraz merged 1 commit intoapache:masterfrom
dave-mccowan:fixlog

Conversation

@dave-mccowan
Copy link
Copy Markdown
Contributor

@dave-mccowan dave-mccowan commented Jul 21, 2024

Change log level from INFO to DEBUG when processing an empty user map during polling. An empty user map is a normal situation for some authenticators (e.g. LDAP) and polling is frequent (1 minute by default.)

Fixes #16766.

Reduce log level to debug for very frequent very normal condition in druid-basic-security.

Description

Fixed the bug ...

Renamed the class ...

Added a forbidden-apis entry ...

Release note

Changed log level from INFO to DEBUG for common condition during polling for authentication cache.


Key changed/added classes in this PR
  • MyFoo
  • OurBar
  • TheirBaz

This PR has:

  • been self-reviewed.
  • a release note entry in the PR description.
  • been tested in a test Druid cluster.

Fixes apache#16766

Change log level from INFO to DEBUG when processing an empty user map
during polling.  An empty user map is a normal situation for some
authenticators (e.g. LDAP) and polling is frequent (1 minute by
default.)
@kfaraz
Copy link
Copy Markdown
Contributor

kfaraz commented Jul 22, 2024

Failing due to coverage. Merging the PR as it is only changing a log line. We can add a test for this class in a separate PR.

@kfaraz kfaraz merged commit 7f7e6ca into apache:master Jul 22, 2024
sreemanamala pushed a commit to sreemanamala/druid that referenced this pull request Aug 6, 2024
Fixes apache#16766

Change log level from INFO to DEBUG when processing an empty user map
during polling.  An empty user map is a normal situation for some
authenticators (e.g. LDAP) and polling is frequent (1 minute by
default.)
@kfaraz kfaraz added this to the 31.0.0 milestone Oct 4, 2024
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.

Non-coordinator services produce excessive logging from druid-basic-security extension

2 participants