Modelbinder errors not transparent

[BennyM]

When null is submitted for a non nullable field you get a null reference exception, no further info is available on what is wrong.

[yentheo]

I will take a look at this.
Any preference?

[BennyM]

Reference variabels should be null. Unsure about value types, default or error. What does MVC do by default?

…

On 26 Aug 2017, at 00:02, yentheo ***@***.***> wrote: I will take a look at this. Any preference? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

[yentheo]

At the moment, if the model binder isn't able to bind everything, he binds nothing.
For example:

public class Request
{
    public string StringValue { get; set; }
    public int IntValue { get; set; }
}

[Route("api/test")]
public void Post([FromBody]Request req)
{
    ...
}

So let's say we post something like this body
{ "stringValue": "value", "intValue": 5 }
The req variable will correctly initialised.
If we post something like { "stringValue": "value", "intValue": null }
The req variable will be null. No warning, no nothing.

[BennyM]

Try to omit it completely so {}

…

On 26 Aug 2017, at 13:36, yentheo ***@***.***> wrote: At the moment, if the model binder isn't able to bind everything, he binds nothing. For example: public class Request { public string StringValue { get; set; } public int IntValue { get; set; } } [Route("api/test")] public void Post([FromBody]Request req) { ... } So let's say we post something like this body { "stringValue": "value", "intValue": 5 } The req variable will correctly initialised. If we post something like { "stringValue": "value", "intValue": null } The req variable will be null. No warning, no nothing. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.