Почему метод async/await в атрибуте никогда не возвращается

Question

Хорошо, так что медведь со мной может потребовать объяснения, у меня есть простой контроллер учетной записи;

[RoutePrefix("api/account")] 
[Authorize] 
[HmacAuthentication] 
public class AccountController : ApiController 
{ 
    public async Task<IHttpActionResult> Register(UserModel userModel) 
    { 
     if (!this.ModelState.IsValid) 
     { 
      return this.BadRequest(this.ModelState); 
     } 

     IdentityResult result = await this._userService.RegisterUser(userModel); 

     var errorResult = this.GetErrorResult(result); 
     if (errorResult != null) 
     { 
      return errorResult; 
     } 

     return this.Ok(); 
    } 
} 

Атрибут HmacAuthentication здесь:

public class HmacAuthenticationAttribute : Attribute, IAuthenticationFilter 
{ 
    public Task AuthenticateAsync(HttpAuthenticationContext context, CancellationToken cancellationToken) 
    { 
     ... 

     var isValid = this.IsValidRequest(req, appId, incomingBase64Signature, nonce, requestTimeStamp); 

     if (isValid.Result) 
     { 
      ... 
     } 
     ... 
    } 

    private async Task<bool> IsValidRequest(
     HttpRequestMessage req, 
     string appId, 
     string incomingBase64Signature, 
     string nonce, 
     string requestTimeStamp) 
    { 
     ... 
     var user = await this.UserService.FindUser(userId); // this never gets a return value 
     ... 
    } 
} 

метод вызывается в UserService это:

public async Task<ApplicationUserModel> FindUser(int id) 
{ 
    var user = await this._userBusiness.FindAsync(id); 
    return this.MapToModel(user); 
} 

и в бизнес-классе это:

public async Task<ApplicationUser> FindAsync(int id) 
{ 
    var result = await this._userManager.FindByIdAsync(id); 
    return result; 
} 

проблема, с которой я сталкиваюсь, заключается в том, что при вызове метода Register активируется атрибут HmacAuthentication и выполняется метод фильтра AuthenticateAsync. Вызов в IsValidRequest для поиска пользователя никогда не получает возвращаемого значения, если я пробую запрос через почтальон, он никогда не завершается.

Может ли кто-нибудь помочь в этом, пожалуйста?

Answer 1

public async Task AuthenticateAsync(HttpAuthenticationContext context, CancellationToken cancellationToken) 
    { 
     ... 

     var isValid = await this.IsValidRequest(req, appId, incomingBase64Signature, nonce, requestTimeStamp); 

     if (isValid) 
     { 
      ... 
     } 
     ... 
    } 

Как компилятор предложил, вы можете использовать только await ключевое слово внутри метода, помеченный как async. Поэтому я обновил подпись для AuthenticateAsync соответственно. Также обратите внимание, что теперь вы можете просто проверить isValid вместо того, чтобы делать isValid.Result, так как значение булева будет доступно за ожидаемой строкой.

Это может быть немного запутанным, поскольку интерфейс для IAuthenticationFilter не указывает async (интерфейсы не могут, они могут только указывать, что метод вернет задание). Разработчик должен определить, будет ли метод просто возвращать задачу или будет поставлять асинхронный запрос, чтобы значения можно было ожидать внутри тела метода.

Answer 2

Принятый ответ правильный, как исправить вашу проблему, но не объясняет , почему это происходит.

Это:

var isValid = this.IsValidRequest(
        req, 
        appId, 
        incomingBase64Signature, 
        nonce, 
        requestTimeStamp); 

if (isValid.Result) 

вызывает ваш код тупиковой. Вы синхронно блокируете метод async. Это происходит потому, что когда компилятор видит метод async, он генерирует машину состояний, которая берет все после первого await и завершает ее как продолжение. Компилятор также видит, есть ли какие-либо контексты синхронизации, и если есть, он пытается перевести продолжение на AspNetSynchronizationContext, который заблокирован вашим вызовом .Result, что фактически вызывает тупик.

Вот почему you shouldn't block on async code