Skip to content

fix: restore real-time event log streaming and fix message content loss#2515

Merged
zzzhangqi merged 1 commit intogoodrain:mainfrom
zzzhangqi:v6.6-0304
Mar 4, 2026
Merged

fix: restore real-time event log streaming and fix message content loss#2515
zzzhangqi merged 1 commit intogoodrain:mainfrom
zzzhangqi:v6.6-0304

Conversation

@zzzhangqi
Copy link
Collaborator

  • Restore message.Content field to ensure log messages have actual content
  • Fix subscription timing: register channel before pushing history to prevent real-time message loss
  • Add rebuildMessageContent to reconstruct Content for file-read messages
  • Increase channel buffer from 10 to 1024 to reduce blocking
  • Apply same fixes to both readEventBarrel and dockerLogEventBarrel

This fixes the issue where build logs were not streaming in real-time, caused by commit 0b96e9b which removed Content field to save memory. The new approach maintains streaming file persistence while ensuring messages can be properly delivered to WebSocket clients.

@zzzhangqi zzzhangqi merged commit 10adc2f into goodrain:main Mar 4, 2026
7 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