Skip to content

Reduce acme challenge log noise - #396

Merged
mholt merged 2 commits into
caddyserver:masterfrom
framer:reduce-acme-challenge-log-noise
Jul 17, 2026
Merged

Reduce acme challenge log noise#396
mholt merged 2 commits into
caddyserver:masterfrom
framer:reduce-acme-challenge-log-noise

Conversation

@danillouz

Copy link
Copy Markdown
Contributor

The "looking up info for HTTP challenge" logs can become very noisy (they produced ~2M logs in 24 hours for us)

This PR changes two "expected" cases to log at DEBUG instead of WARN:

  • Missing ACME challenges
  • Challenge lookups canceled together with their HTTP request

(see the individual commit messages for more detail)

FWIW: these changes have been running in our fork for the last week without any unexpected behavior AFAICS

The "looking up info for HTTP challenge" logs can become very noisy
(they produced ~2M logs in 24 hours for us).

Requests can come from external ACME clients and pre-checks for
challenges not initiated by the server. So treat this case
(where no issuer has stored a matching token) as DEBUG to reduce
noise.

Note that actionable failures will remain visible:

- Storage lookup failures: WARN
- Empty or corrupt challenge data: WARN
- Certificate issuance/renewal failures: visible via existing error logs
A big chunk of the noisy "looking up info for HTTP challenge"
logs are "context canceled".

So also log these as DEBUG, but only do so when both the
lookup error and HTTP request context are canceled.

@mholt mholt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, I think this is an acceptable change, and probably makes the most sense. Thank you! And thanks for your team's sponsorship.

@mholt
mholt merged commit 38cdd62 into caddyserver:master Jul 17, 2026
6 checks passed
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.

2 participants