Skip to content

feat: log ignored endpoints at debug level - #25

Closed
jvmakine wants to merge 1 commit into
mainfrom
feat/demote-ignored-endpoint-logs
Closed

jvmakine wants to merge 1 commit into
mainfrom
feat/demote-ignored-endpoint-logs

Conversation

@jvmakine

@jvmakine jvmakine commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

Health and status endpoints declared with ingress.ignore (such as GET /_status) are polled constantly. Their request and comparison lines were logged at info level, so they flooded the default output and buried meaningful traffic:

ingress INF HTTP request [ingress_received] method=GET path=/_status status=200
ingress INF Response comparison completed [correlation] outcome=skipped path=/_status reason=endpoint "GET /_status" is ignored

This logs both lines at debug when the request matches an ignored endpoint, leaving all other traffic at info. The ignore decision needs only the method and path, so the ingress logging middleware now takes an optional predicate backed by the comparator, and the comparator logs its own completion line at debug when the endpoint is ignored. Start the proxy with --log-level=debug to see them again.

🤖 Generated with Claude Code

Health and status endpoints declared with ingress.ignore are polled
constantly, so their request and comparison lines flooded the info-level
output and buried meaningful traffic. Log both lines at debug when the
endpoint is ignored, leaving all other traffic at info.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@jvmakine
jvmakine requested a review from alecthomas as a code owner October 7, 2026 04:34
@jvmakine jvmakine closed this Oct 7, 2026
@jvmakine
jvmakine deleted the feat/demote-ignored-endpoint-logs branch October 7, 2026 04:36
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.

1 participant