Skip to content

fix: don't send body on GET/HEAD with json_body auth#88

Merged
matthiastjong merged 1 commit into
mainfrom
fix/json-body-get-request
May 18, 2026
Merged

fix: don't send body on GET/HEAD with json_body auth#88
matthiastjong merged 1 commit into
mainfrom
fix/json-body-get-request

Conversation

@matthiastjong
Copy link
Copy Markdown
Owner

Summary

  • Fix HTTP spec violation: json_body auth was attaching a body to GET/HEAD requests
  • Now only merges stored JSON into the body for POST/PUT/PATCH/DELETE
  • Also skips setting Content-Type: application/json on GET/HEAD

Test plan

  • New test: json_body does not send body on GET requests
  • 18/18 gateway tests passing

🤖 Generated with Claude Code

GET/HEAD requests with a json_body auth method would fail because the
gateway was attaching a body, violating the HTTP spec.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@matthiastjong matthiastjong merged commit c1495bc into main May 18, 2026
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