Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Intercept bad requests before reaching controller in ASP.NET Core

I have a logic to apply in case the request received is a BadRequest, to do this I have created a filter:

public class ValidateModelAttribute : ActionFilterAttribute
{
    public override void OnActionExecuting(ActionExecutingContext context)
    {
        if (!context.ModelState.IsValid)
        {
            // Apply logic
        }
    }
}

In Startup:

public void ConfigureServices(IServiceCollection services)
{
    services.AddMvc(options => { options.Filters.Add<ValidateModelAttribute>(); });
}

Controller:

[Route("api/[controller]")]
[ApiController]
public class VerifyController : ControllerBase
{
    [Route("test")]
    [HttpPost]
    [ValidateModel]
    public ActionResult<Guid> validationTest(PersonalInfo personalInfo)
    {
        return null;
    }
}

Model:

public class PersonalInfo
{
    public string FirstName { get; set; }
    [RegularExpression("\\d{4}-?\\d{2}-?\\d{2}", ErrorMessage = "Date must be properly formatted according to ISO 8601")]
    public string BirthDate { get; set; }
}

The thing is when I put a break point on the line:

if (!context.ModelState.IsValid)

execution reaches this line only if the request I send is valid. Why it is not passing the filter if I send a bad request?

like image 896
Yahya Hussein Avatar asked Aug 16 '18 06:08

Yahya Hussein


People also ask

Can ASP.NET Core handle many requests?

ASP.NET Core apps should be designed to process many requests simultaneously. Asynchronous APIs allow a small pool of threads to handle thousands of concurrent requests by not waiting on blocking calls. Rather than waiting on a long-running synchronous task to complete, the thread can work on another request.

How does Web API handle global exception in .NET Core?

The middleware UseExceptionHandler can be used to handle exceptions globally. You can get all the details of the exception object (Stack Trace, Inner exception, message etc..) and display them on-screen. You can implement like this.


1 Answers

The [ApiController] attribute that you've applied to your controller adds Automatic HTTP 400 Responses to the MVC pipeline, which means that your custom filter and action aren't executed if ModelState is invalid.

I see a few options for affecting how this works:

  1. Remove the [ApiController] attribute

    Although you can just remove the [ApiController] attribute, this would also cause the loss of some of the other features it provides, such as Binding source parameter inference.

  2. Disable only the Automatic HTTP 400 Responses

    Here's an example from the docs that shows how to disable just this feature:

    services.AddControllers()
        .ConfigureApiBehaviorOptions(options =>
        {
            // ...
            options.SuppressModelStateInvalidFilter = true;
            // ...
        }
    

    This code goes inside of your Startup's ConfigureServices method.

  3. Customise the automatic response that gets generated

    If you just want to provide a custom response to the caller, you can customise what gets returned. I've already described how this works in another answer, here.

like image 173
Kirk Larkin Avatar answered Sep 20 '22 12:09

Kirk Larkin