Async OnActionExecuting in ASP.NET Core's ActionFilterAttribute

Question

ASP.NET Core's ActionFilterAttribute has these:

public virtual void OnActionExecuting(ActionExecutingContext context);
public virtual void OnActionExecuted(ActionExecutedContext context);
public virtual Task OnActionExecutionAsync(ActionExecutingContext context, ActionExecutionDelegate next);

I need an async version of OnActionExecuting, which doesn't exist.

However I have a feeling that I can use OnActionExecutionAsync instead, as it also has an argument of ActionExecutingContext.

Am I correct that despite the name, they trigger at the same point in the process?

Also, what do I need to do with the next argument? Once I'm done with my stuff, do I simply need to call await next()?

Is that it? I'm unsure as I can't find docs for this.

jeroenh · Accepted Answer · 2017-05-15 13:11:20Z

118

Asynchronous filters work a bit differently: first execute code that must be executed before the action, call next() for the actual logic, finally add code to be executed after the action.

public async Task OnActionExecutionAsync(ActionExecutingContext context, 
                                         ActionExecutionDelegate next)
{

    // logic before action goes here

    await next(); // the actual action

    // logic after the action goes here
}

The documentation is here: https://learn.microsoft.com/en-us/aspnet/core/mvc/controllers/filters#implementation

edited May 15, 2017 at 13:11

answered May 15, 2017 at 12:49

jeroenh

26.9k11 gold badges77 silver badges107 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

grokky Over a year ago

Actually it's in here in the second code block

juunas Over a year ago

@grokky The base implementation checks the arguments and calls the two other methods. You don't need to call it if you are not overriding the two other methods.

juunas Over a year ago

@grokky You can check the source code here: github.com/aspnet/Mvc/blob/dev/src/…

jsgoupil Over a year ago

For those searching if the delegate threw an exception, take the result from var result = await next(); then check for result.Exception

A. Nadjar · Accepted Answer · 2019-09-23 06:47:07Z

Asynchronous filters always take precedence over the synchronous filter implementations.

According to the Docs:

It's advised to implement either the synchronous or the async version of a filter interface, not both. The runtime checks first to see if the filter implements the async interface, and if so, it calls that. If not, it calls the synchronous interface's method(s). If both asynchronous and synchronous interfaces are implemented in one class, only the async method is called.

However, you can manage to have both. For instance:

public class TimestampFilter : IActionFilter, IAsyncActionFilter 
{    
    public void OnActionExecuting(ActionExecutingContext context)    
    {         
        context.ActionDescriptor.RouteValues["timestamp"] = DateTime.Now.ToString();    
    }

    public void OnActionExecuted(ActionExecutedContext context)    
    {         
        var ts = DateTime.Parse(context.ActionDescriptor. RouteValues["timestamp"]).AddHours(1).ToString();        
        context.HttpContext.Response.Headers["X-EXPIRY-TIMESTAMP"] = ts;    
    }

     public async Task OnActionExecutionAsync(ActionExecutingContext context, ActionExecutionDelegate next)    
    {        
        this.OnActionExecuting(context);        
        var resultContext = await next();
        this.OnActionExecuted(resultContext);    
    }
 }

XDS · Accepted Answer · 2024-03-06 10:41:01Z

1

An ever better pattern:

public override async Task OnActionExecutionAsync(
    ActionExecutingContext context, 
    ActionExecutionDelegate next)
{
    try
    {
        // logic before action goes here
    }
    finally
    {
        // await base.OnActionExecutionAsync(context, next); //dont
        await next(); // the actual action

        // logic after the action goes here
    }
}

edited Mar 6, 2024 at 10:41

answered Mar 4, 2021 at 7:47

XDS

4,2944 gold badges43 silver badges72 bronze badges

2 Comments

Adam K Over a year ago

This causes actions to be invoked multiple times per request. do not use this method.

XDS Over a year ago

I have improved the implementation to address your concerns @AdamK

Collectives™ on Stack Overflow

Async OnActionExecuting in ASP.NET Core's ActionFilterAttribute

3 Answers 3

4 Comments

Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

4 Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related