fix(net/ghttp): report invalid json body errors

[puneetdixit200]

Summary:

• set request errors when JSON-looking request bodies fail to parse under application/json
• propagate parse errors through struct binding for request, query, and form params
• add a regression covering malformed JSON request bodies

Tests:

• GOMODCACHE=/tmp/gf-gomodcache GOCACHE=/tmp/gf-gocache go test ./net/ghttp -run Test_ParamsInvalidJsonReportsParseError -count=1
• GOMODCACHE=/tmp/gf-gomodcache GOCACHE=/tmp/gf-gocache go test ./net/ghttp -run 'Test_Params(InvalidJsonReportsParseError|TagDefault|TagIn|_Valid|_Parse|_ParseForm|_ParseQuery)|Test_Params_Get(Query|QueryMap|RequestMap)|Test_Issue4093|Test_Issue4227' -count=1\n- GOMODCACHE=/tmp/gf-gomodcache GOCACHE=/tmp/gf-gocache go test ./net/gclient -run Test_Client_Chain_ContentJson -count=1\n- GOMODCACHE=/tmp/gf-gomodcache GOCACHE=/tmp/gf-gocache go test ./net/ghttp -run Test_ParamsInvalidJsonReportsParseError -race -count=1\n- git diff --check\n\nFixes No error will be reported for illegal JSON requests #4087

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR improves error propagation for request parameter binding when a request has invalid JSON, and adds a unit test to verify that JSON parse errors are reported instead of falling back to struct validation.

Changes:

• Propagate request parsing errors earlier in Get*Struct helpers (Request, Query, Form).
• Detect JSON body unmarshal failures and surface an invalid-parameter error when Content-Type indicates JSON.
• Add a unit test covering invalid JSON input and expected error messaging.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
net/ghttp/ghttp_z_unit_feature_request_param_test.go	Adds a controller + test asserting invalid JSON yields a JSON parse error response
net/ghttp/ghttp_request_param_request.go	Early-return r.GetError() from request-struct binding
net/ghttp/ghttp_request_param_query.go	Early-return r.GetError() from query-struct binding
net/ghttp/ghttp_request_param_form.go	Early-return r.GetError() from form-struct binding
net/ghttp/ghttp_request_param.go	Sets request error when JSON unmarshal fails for JSON Content-Type

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.