Skip to content

fix: Reduce log verbosity for element state changes and WebSocket lag - #38

Merged
srperens merged 1 commit into
mainfrom
fix/reduce-log-verbosity
Nov 24, 2025
Merged

fix: Reduce log verbosity for element state changes and WebSocket lag#38
srperens merged 1 commit into
mainfrom
fix/reduce-log-verbosity

Conversation

@srperens

Copy link
Copy Markdown
Collaborator

Summary

  • Change element state change logging from info! to debug! level (pipeline-level state changes remain at info for visibility)
  • Change "Client is lagging" WebSocket warning to debug! level (expected behavior during pipeline startup bursts)
  • Remove unused warn import from websocket.rs

Problem

During pipeline startup, hundreds of log lines were generated:

  • Every GStreamer element state transition (Null->Ready->Paused->Playing) was logged at INFO level
  • WebSocket clients received "lagging" warnings during startup bursts when they couldn't keep up with the flood of events

Solution

  • Element state changes: moved to DEBUG level (pipeline-level changes remain at INFO)
  • WebSocket lag warnings: moved to DEBUG level since it's not an error condition

Before/After

Before: Hundreds of state changed logs + dozens of lagging warnings per pipeline start
After: Only 3 pipeline state transitions logged per flow (Null->Ready->Paused->Playing)

Test plan

  • Run RUST_LOG=info cargo run -p strom-backend and verify clean startup logs
  • Verify pipeline state changes still logged at INFO level
  • Verify no "Client is lagging" warnings at INFO level
  • Pre-commit hooks pass (fmt + clippy)

🤖 Generated with Claude Code

- Change element state change logging from info to debug level
  (pipeline-level state changes remain at info for visibility)
- Change "Client is lagging" WebSocket warning to debug level
  (this is expected behavior during pipeline startup bursts)
- Remove unused `warn` import from websocket.rs

Before: Hundreds of state change logs + many lagging warnings per pipeline start
After: Only 3 pipeline state transitions logged per flow (Null->Ready->Paused->Playing)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@srperens
srperens merged commit 2af438f into main Nov 24, 2025
4 of 5 checks passed
@srperens
srperens deleted the fix/reduce-log-verbosity branch November 25, 2025 13:56
birme pushed a commit to eyevinn-osaas/strom that referenced this pull request Feb 2, 2026
fix: Reduce log verbosity for element state changes and WebSocket lag
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