Skip to content

Defer WebSocket dry-run body materialization#688

Merged
ryanfowler merged 1 commit into
mainfrom
fix-websocket-dryrun-body-read
Jun 1, 2026
Merged

Defer WebSocket dry-run body materialization#688
ryanfowler merged 1 commit into
mainfrom
fix-websocket-dryrun-body-read

Conversation

@ryanfowler

Copy link
Copy Markdown
Owner

Summary

  • Move WebSocket initial-message construction until after the dry-run early return so --dry-run no longer reads stdin or large request bodies.
  • Add a regression test that keeps stdin open with -d @- and verifies dry-run exits without blocking.

Testing

  • cargo fmt
  • cargo clippy --locked --all-targets --all-features -- -D warnings
  • cargo test --all-features --test websocket websocket_dry_run
  • cargo test --all-features

@ryanfowler ryanfowler enabled auto-merge June 1, 2026 04:14
@ryanfowler ryanfowler merged commit e061c16 into main Jun 1, 2026
4 checks passed
@ryanfowler ryanfowler deleted the fix-websocket-dryrun-body-read branch June 1, 2026 04:18
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