Skip to content

Lower log level in HttpServerConnection#10882

Open
mcodato wants to merge 1 commit into
Icinga:masterfrom
WuerthPhoenix:lower-log-level-http-server-connection
Open

Lower log level in HttpServerConnection#10882
mcodato wants to merge 1 commit into
Icinga:masterfrom
WuerthPhoenix:lower-log-level-http-server-connection

Conversation

@mcodato

@mcodato mcodato commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Change the log level from LogInformation to LogNotice to reduce log noise in large environments. With many checks running, info-level messages can generate excessive log volume, making it harder to identify relevant events and understand what is happening.

@cla-bot cla-bot Bot added the cla/signed label Jun 16, 2026
@julianbrost

Copy link
Copy Markdown
Member

From my experience with handling past support cases, the HTTP requests are often some of the useful information included in the logs. So doing that change would take this away by default. Also, the notice level unfortunately is already pretty verbose is some places where it's almost a debug log already, making it harder to convince people to enable it. Therefor, I'm currently not really in favor of changing this as proposed.

What rate of requests and amount of resulting logs are we talking about here that this becomes problematic?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants