Using Required and JsonRequired in ASP.NET Core Model Binding with JSON body

问题

I'm using ASP.NET Core 2.0, and I have a request object annotated like this:

public class MyRequest
{
    [Required]
    public Guid Id { get; set; }

    [Required]
    public DateTime EndDateTimeUtc { get; set; }

    [Required]
    public DateTime StartDateTimeUtc { get; set; }
}

And in my controller:

public async Task<IActionResult> HandleRequest([FromBody] MyRequest request)
{ /* ... */ }

I noticed an issue with model binding: When I send a request containing the header Content-Type set to application/json and a an empty body, as I expect, the request in my controller is null and ModelState.IsValid is false.

But when I have a body like this:

{
  "hello": "a-string-value!"
}

my request is NOT null, it has default values for everything, and ModelState.IsValid is true

This is happening of course while I'm missing all the Required properties, and the only existing one's name doesn't match a property there (even the type for this single parameter is string, which doesn't match any type on my model).

So in a way, those Required attributes seem to be working if there's nothing in my request, but they don't do anything if my request is not empty!

As I was preparing this question, I noticed that there's also a JsonRequired attribute, and it seems to take care of the properties being present.

So, what's the difference between Required and JsonRequired?

回答1:

For correct work of Required attribute, you should make the properties nullable:

public class MyRequest
{
    [Required]
    public Guid? Id { get; set; }

    [Required]
    public DateTime? EndDateTimeUtc { get; set; }

    [Required]
    public DateTime? StartDateTimeUtc { get; set; }
}

Now if you send request with missing Id, EndDateTimeUtc or StartDateTimeUtc, corresponding field will be set to null, ModelState.IsValid will be set to false and ModelState will contain error(s) description, e.g. The EndDateTimeUtc field is required.

JsonRequired attribute is specific to JSON.Net. It plays during deserialization, while Required attribute (as other attributes from System.ComponentModel.DataAnnotations namespace) plays after model is deserialized, during model validation. If JsonRequired attribute is violated, the model will not be deserialized at all and corresponding action parameter will be set to null.

The main reason why you should prefer Required attribute over JsonRequired is that JsonRequired will not work for other content types (like XML). Required in its turn is universal since it's applied after the model is deserialized.

回答2:

When you use [FromBody] as binding source, the Model properties will get default values and [BindRequired] will be ignored. There is a related issue on "Problems with Parameter Validation".

In this case is better to use [JsonRequired] instead of [BindRequired] to enforce binding properties.

Note that [JsonRequired] affects serialization and deserialization both.

来源：https://stackoverflow.com/questions/49237767/using-required-and-jsonrequired-in-asp-net-core-model-binding-with-json-body

标签

json.net

asp.net-core-2.0

model-binding