Skip to content

Bound WebSocket text stdin lines and document binary fallback#699

Merged
ryanfowler merged 1 commit into
mainfrom
fix-websocket-stdin-line-limit
Jun 2, 2026
Merged

Bound WebSocket text stdin lines and document binary fallback#699
ryanfowler merged 1 commit into
mainfrom
fix-websocket-stdin-line-limit

Conversation

@ryanfowler

Copy link
Copy Markdown
Owner

Summary

  • Cap WebSocket text/auto stdin lines at 16 MiB before a newline
  • Return a clear error that points users to --ws-message-mode binary for raw streams
  • Add unit coverage for the overflow case, CRLF handling at the limit, and binary chunk streaming
  • Document the stdin limit in the WebSocket docs and CLI reference

Testing

  • cargo fmt
  • cargo clippy --locked --all-targets --all-features -- -D warnings
  • cargo test --all-features
  • cargo test --all-features --test cli --test formatting --test grpc --test http --test network --test terminal --test update --test websocket -- --test-threads=1

@ryanfowler ryanfowler merged commit d3e866d into main Jun 2, 2026
4 checks passed
@ryanfowler ryanfowler deleted the fix-websocket-stdin-line-limit branch June 2, 2026 16:41
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