Add HTTP QUERY method support#1319
Conversation
9c386db to
48814f7
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1319 +/- ##
==========================================
+ Coverage 87.67% 87.82% +0.15%
==========================================
Files 30 30
Lines 11739 11769 +30
==========================================
+ Hits 10292 10336 +44
+ Misses 1447 1433 -14 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Reviewed against RFC 10008 — the implementation looks correct and complete. The redirect matrix is the subtle part and it's exactly right: QUERY is preserved on 301/302/307/308 and only rewritten to GET on 303, matching §2.5 — including the easy-to-miss note that "the exceptions for redirecting a POST as a GET after a 301 or 302 do not apply to QUERY." (Verified the preserved-method path carries the body via One QUERY-specific nuance worth considering (non-blocking): §2/§2.1 makes [findings generated by claude] |
def72cd to
7833e08
Compare
7833e08 to
79e4818
Compare
Summary
Adds first-class support for the RFC 10008 HTTP
QUERYmethod across HTTP.jl's client and server-facing APIs.HTTP.query(...),HTTP.query(client, ...), andHTTP.@clienthelper support without adding a new bare export.QUERYas safe/idempotent for high-level retries and stale reused-connection transport retries.QUERYmethod and replayable body across301,302,307, and308redirects while keeping303rewrite behavior toGET.Accept-Queryheader and documentsQUERYrequest usage.Validation
julia --project=. test/runtests.jljulia --project=docs docs/make.jlReseau1.3.1 checkout:http_core_tests.jl,http_handlers_tests.jl,http_client_tests.jl,http_client_transport_tests.jlNote: the local checkout has an untracked manifest pointing at an older dirty
~/.julia/dev/Reseau, so the final focused verification used a temporary clean Reseau 1.3.1 checkout to match the currentmastercompat.Co-authored by Codex