Skip to content

fix(http): skip decoding metadata-only responses - #1170

Merged
ryanfowler merged 1 commit into
mainfrom
fix/metadata-only-response-decoding
Aug 30, 2026
Merged

fix(http): skip decoding metadata-only responses#1170
ryanfowler merged 1 commit into
mainfrom
fix/metadata-only-response-decoding

Conversation

@ryanfowler

Copy link
Copy Markdown
Owner

Summary

  • avoid constructing content decoders for responses that HTTP semantics define as content-free
  • cover HEAD, informational responses, 204, 205, 304, and successful CONNECT tunnels
  • preserve representation and wire-length metadata without producing spurious decoder EOF errors

Testing

  • go test -parallel 8 ./...
  • go build .
  • go mod tidy && go mod verify
  • staticcheck ./...

@ryanfowler
ryanfowler enabled auto-merge August 30, 2026 21:34
@ryanfowler
ryanfowler merged commit 675623a into main Aug 30, 2026
12 checks passed
@ryanfowler
ryanfowler deleted the fix/metadata-only-response-decoding branch August 30, 2026 21:36
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