Skip to content

Reduce log message verbosity - #341

Merged
vickenty merged 1 commit into
masterfrom
vickenty/rlv
Sep 10, 2026
Merged

Reduce log message verbosity#341
vickenty merged 1 commit into
masterfrom
vickenty/rlv

Conversation

@vickenty

Copy link
Copy Markdown
Contributor

No description provided.

@vickenty
vickenty marked this pull request as ready for review September 10, 2026 10:28
@vickenty
vickenty requested a review from a team as a code owner September 10, 2026 10:28
@vickenty
vickenty merged commit fc48b2d into master Sep 10, 2026
23 checks passed
@vickenty
vickenty deleted the vickenty/rlv branch September 10, 2026 10:30

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c5d3133d8b

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines +148 to +150
Level.FINER,
"sending {0} bytes to {1}: {2} {3}",
new Object[] {payload.bytes.length, url, res.statusCode(), res.body()});

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep failed HTTP responses visible

When the endpoint returns 400 or a 5xx response, this unconditional FINER record is filtered out by the usual INFO logging threshold even though handleResponse immediately drops a 400 payload or retries and eventually drops other non-200 payloads. Telemetry retains only the status counters, not the actionable response body, so server-side rejections become silent while transport failures still log at WARNING; use the reduced level only for successful responses and retain a visible level for non-200 responses.

Useful? React with 👍 / 👎.

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