fix(binding): When there is a body parameter that is required, if the…

…re is no body, an error is returned

Change-Id: Iacc4e379d1a1aa25b23f972d4abbea8f95a3d463

binding/bind.go

@@ -143,6 +143,9 @@ func (b *Binding) bind(structPointer interface{}, req *http.Request, pathParams
 			case form, json, protobuf:
 				if info.paramIn == in(bodyCodec) {
 					found, err = param.bindOrRequireBody(info, expr, bodyCodec, bodyString, postForm)
+				} else if info.required {
+					found = false
+					err = info.requiredError
 				}
 			case auto:
 				// Try bind parameters from the body when the request has body,

binding/bind_test.go

@@ -71,6 +71,19 @@ func TestQueryString(t *testing.T) {
 	assert.Equal(t, (*string)(nil), recv.Z)
 }
 
+func TestGetBody(t *testing.T) {
+	type Recv struct {
+		X **struct {
+			E string `json:"e,required"`
+		}
+	}
+	req := newRequest("http://localhost:8080/", nil, nil, nil)
+	recv := new(Recv)
+	binder := binding.New(nil)
+	err := binder.BindAndValidate(recv, req, nil)
+	assert.Error(t, &binding.Error{ErrType: "binding", FailField: "X.e", Msg: "missing required parameter"}, err)
+}
+
 func TestQueryNum(t *testing.T) {
 	type Recv struct {
 		X **struct {