blackteay/gin
3
0
Fork 0
mirror of https://github.com/gin-gonic/gin.git synced 2026-04-29 23:23:18 +08:00
Code Issues Projects Releases Wiki Activity

quick fix to skip binding body without content type

Browse Source
This commit is contained in:
takanuva15 2026-03-30 17:29:21 -04:00
parent 64a0c537f7
commit 79c32ff4aa
2 changed files with 16 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
binding/all.go
View File

@ -41,6 +41,12 @@ func (allBinding) BindMany(req *http.Request, uriParams map[string][]string, obj
if contentTypeLastIdx != -1 { if contentTypeLastIdx != -1 {
contentType = contentType[:contentTypeLastIdx] contentType = contentType[:contentTypeLastIdx]
} }
// if no Content-Type, assume request has no body. This avoids binding query params again in binding.Default
if contentType == "" {
return validate(obj)
}
b := Default(req.Method, contentType) b := Default(req.Method, contentType)
// final validation done by whatever binding is selected here // final validation done by whatever binding is selected here
return b.Bind(req, obj) return b.Bind(req, obj)

6
context.go
View File

@ -810,7 +810,8 @@ func (c *Context) BindUri(obj any) error {
// Note: // Note:
// - Caller must tag struct fields appropriately for the desired binding (eg `header:"xxx"` vs `uri:"xxx"`) // - Caller must tag struct fields appropriately for the desired binding (eg `header:"xxx"` vs `uri:"xxx"`)
// - Caller must ensure no duplication between field names (else use separate binding engines instead) // - Caller must ensure no duplication between field names (else use separate binding engines instead)
// - Caller must provide Content-Type header to select the correct body binding (eg "application/json" for JSON binding) // - Caller must provide Content-Type header to select the correct body binding (eg "application/json" for JSON binding).
// A request without Content-Type will result in no body binding
// - Binding validation tags are verified after all request parts have been bound // - Binding validation tags are verified after all request parts have been bound
func (c *Context) BindAll(obj any) error { func (c *Context) BindAll(obj any) error {
err := c.ShouldBindAll(obj) err := c.ShouldBindAll(obj)
@ -947,7 +948,8 @@ func (c *Context) ShouldBindUri(obj any) error {
// Note: // Note:
// - Caller must tag struct fields appropriately for the desired binding (eg `header:"xxx"` vs `uri:"xxx"`) // - Caller must tag struct fields appropriately for the desired binding (eg `header:"xxx"` vs `uri:"xxx"`)
// - Caller must ensure no duplication between field names (else use separate binding engines instead) // - Caller must ensure no duplication between field names (else use separate binding engines instead)
// - Caller must provide Content-Type header to select the correct body binding (eg "application/json" for JSON binding) // - Caller must provide Content-Type header to select the correct body binding (eg "application/json" for JSON binding).
// A request without Content-Type will result in no body binding
// - Binding validation tags are verified after all request parts have been bound // - Binding validation tags are verified after all request parts have been bound
func (c *Context) ShouldBindAll(obj any) error { func (c *Context) ShouldBindAll(obj any) error {
uriParams := make(map[string][]string, len(c.Params)) uriParams := make(map[string][]string, len(c.Params))