Skip to content

Bound dry-run request body previews#717

Merged
ryanfowler merged 1 commit into
mainfrom
fix-dryrun-body-preview
Jun 5, 2026
Merged

Bound dry-run request body previews#717
ryanfowler merged 1 commit into
mainfrom
fix-dryrun-body-preview

Conversation

@ryanfowler

Copy link
Copy Markdown
Owner

Summary

  • Added a bounded dry-run preview path that prints at most 1024 bytes and emits a truncation warning when the preview is cut off.
  • Removed the old dry-run behavior that could fully materialize stdin, gRPC, file, and multipart bodies just to inspect them.
  • Added multipart preview support that reads file parts only as far as needed for the preview cap.
  • Added regression coverage for large @file, @-, and multipart file bodies.

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 enabled auto-merge June 5, 2026 20:04
@ryanfowler ryanfowler merged commit 3aca8e9 into main Jun 5, 2026
4 checks passed
@ryanfowler ryanfowler deleted the fix-dryrun-body-preview branch June 5, 2026 20:07
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