fix: shouldBind instead of Bind to prevent abort (#830)

Co-authored-by: Jeroen de Kok <jeroen@weave.nl>
deepmap · Nov 1, 2022 · 050c4bf · 050c4bf
1 parent 7dead8e
commit 050c4bf
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/internal/test/strict-server/gin/server.gen.go b/internal/test/strict-server/gin/server.gen.go
diff --git a/pkg/codegen/templates/strict/strict-gin.tmpl b/pkg/codegen/templates/strict/strict-gin.tmpl
@@ -35,7 +35,8 @@ type strictHandler struct {
             {{if $multipleBodies}}if strings.HasPrefix(ctx.GetHeader("Content-Type"), "{{.ContentType}}") { {{end}}
                 {{if eq .NameTag "JSON" -}}
                     var body {{$opid}}{{.NameTag}}RequestBody
-                    if err := ctx.Bind(&body); err != nil {
+                    if err := ctx.ShouldBind(&body); err != nil {
+                        ctx.Status(http.StatusBadRequest)
                         ctx.Error(err)
                         return
                     }