asp.net core 2.0 model validation not validating data

Question

I am using ASP.NET Core 2.0 default model validation and it seems not working with the ModelState.IsValid always true despite having wrong data in model.

[HttpPost("abc")]
public async Task<IActionResult> Abc([FromBody]AbcViewModel model)
{
    if (!ModelState.IsValid) { return BadRequest(ModelState); }
    ...
}

public class AbcViewModel
{
    [Required(ErrorMessage = "Id is required")]
    [Range(100, int.MaxValue, ErrorMessage = "Invalid Id")]
    public int Id { get; set; }

    public bool Status { get; set; }
}

When I post data from Angular app, the values are mapping to model correctly but if Id is "0" or less than 100, both the Required and Range validators aren't working and ModelState.IsValid is always true. What I am missing?

You need to set the Id to nullable else the value will be defaulted to 0 even if the client don't send an Id in the request. — Marcus Höglund, Aug 11 '18 at 11:34
I've copied your code exactly, and I can't reproduce your problem. One thing to note, however: `Required` checks that a value won't be `null`, an empty string, or whitespace. In your case, `Id` is a non-nullable `int`, so that will always provide a value. — John H, Aug 11 '18 at 11:34
Alright I got the case for required as it should be nullable to work. But what about the Range attribute, why it's not working? — Ali Shahzad, Aug 11 '18 at 11:38
@AliShahzad, same for me, `Range` works as expected - when `Id` is outside of range or not supplied (zero by default) i get a `400 BadRequest` POST /api/values/abc HTTP/1.1 Host: localhost:61154 Content-Type: application/json Cache-Control: no-cache Postman-Token: 62024574-776a-4e1b-93b7-6a7bb78ecfca { "Id": 1 } 400 Bad Request {"Id":["Invalid Id"]} — ironstone13, Aug 11 '18 at 13:08
@ironstone13 But it's not working for me which I guess I'm missing something — Ali Shahzad, Aug 13 '18 at 04:36

score 11 · Answer 1 · answered Mar 01 '19 at 22:59

11

If you're using services.AddMvcCore(), then you need to explicitly set up your application to perform validation with data annotations:

public void ConfigureServices(IServiceCollection services)
{
    services.AddMvcCore()
         .AddDataAnnotations()
         /* etc. */;
}

answered Mar 01 '19 at 22:59

Collin K

15,277
1
27
22

Thanks @Collin that fixed a similar problem that we were experiencing. – Rob Mar 18 '19 at 13:30

score 1 · Answer 2 · answered Mar 02 '19 at 05:10

I used the same model property that you defined in the model and facing the same issue. I made one change in the property and defined the DataMember attribute on the property like:

[Required(ErrorMessage = "Id is required")]
[Range(100, int.MaxValue, ErrorMessage = "Invalid Id")]
[DataMember(Name = "Id")]
public int Id { get; set; }

It's working as expected, validating the range values. Try this, hope it will resolve the issue that you are facing.

score 1 · Answer 3 · answered Sep 09 '19 at 08:05

1

I also fought with this problem and what helped in my case was adding this in ConfigureServices:

services.AddMvc(opt=> {
    opt.AllowValidatingTopLevelNodes = true;
});

Just be carefull about the outcome - if you want to get automatic BadRequest you have to use [ApiController] attribute. Otherwise you have to check ModelState.IsValid property.

answered Sep 09 '19 at 08:05

Geralt

142
1
8

`AllowValidatingTopLevelNodes` is obsolete from 2.2 and not found in 3.1. Any idea how to use it in 3.1? – liang Nov 10 '20 at 03:24
According to https://github.com/dotnet/AspNetCore.Docs/issues/18438#issuecomment-638494816 validation of top level nodes is enabled by default when using any of mentioned services. – Geralt Nov 11 '20 at 07:39

asp.net core 2.0 model validation not validating data

3 Answers3