From a9b274943a9edb0557b3df116e210cbd8da789d4 Mon Sep 17 00:00:00 2001 From: Ryan Fowler Date: Wed, 14 Jan 2026 20:31:03 -0800 Subject: [PATCH] Avoid printing binary request body to stderr during dry-run --- internal/fetch/fetch.go | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/internal/fetch/fetch.go b/internal/fetch/fetch.go index ca12105..cc22aed 100644 --- a/internal/fetch/fetch.go +++ b/internal/fetch/fetch.go @@ -146,8 +146,18 @@ func fetch(ctx context.Context, r *Request) (int, error) { errPrinter.WriteString("\n") errPrinter.Flush() - _, err = io.Copy(os.Stderr, req.Body) - return 0, err + ok, r, err := isPrintable(req.Body) + if err != nil { + return 0, err + } + if ok { + _, err = io.Copy(os.Stderr, r) + return 0, err + } + + msg := "the request body appears to be binary" + core.WriteWarningMsg(errPrinter, msg) + return 0, nil } errPrinter.WriteString("\n")