Skip to content

Reduce log verbosity: move per-request chatter to debug/trace#1

Merged
dmonagle merged 1 commit into
mainfrom
claude/fervent-knuth
Apr 5, 2026
Merged

Reduce log verbosity: move per-request chatter to debug/trace#1
dmonagle merged 1 commit into
mainfrom
claude/fervent-knuth

Conversation

@dmonagle
Copy link
Copy Markdown
Contributor

@dmonagle dmonagle commented Apr 5, 2026

Move verbose per-request logging from info to debug and trace levels to reduce noise in production logs:

  • CharacteristicProtocol.swift: Downgrade GET /accessories and PUT /characteristics logging
    • GET /accessories response body moved to trace level
    • PUT /characteristics request details and per-characteristic operations moved to debug
    • Decode errors remain at warning level
  • HAPBridge.swift: Downgrade event subscription and notification logging to debug level

These changes keep important information available for debugging while reducing log spam during normal operation.

Routine HAP session activity (GET /accessories polls, PUT /characteristics
subscription batches, event subscribe/notify) was logged at info, flooding
operator logs on every Home.app reconnect. Move these to debug or trace so
info shows only lifecycle events (server start, new connections, pairing).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@dmonagle dmonagle merged commit c1df487 into main Apr 5, 2026
2 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.

1 participant