ASP.NET contrôleur: module ou manipulateur asynchrone terminé alors qu'une opération asynchrone était toujours en attente

j'ai rencontré un problème connexe. Un client utilise une Interface qui retourne la tâche et est implémentée avec async.

dans Visual Studio 2015, la méthode client qui est async et qui n'utilise pas le mot-clé attente lors de l'invocation de la méthode ne reçoit aucun avertissement ou erreur, le code se compile proprement. Une condition de race est promue à la production.

ASP.NET considère qu'il est illégal de commencer une "opération asynchrone" liée à son SynchronizationContext et de retourner un ActionResult avant que toutes les opérations commencées soient terminées. Toutes les méthodes async s'enregistrent comme des "opérations asynchrones", vous devez donc vous assurer que tous ces appels qui se lient à la ASP.NET SynchronizationContext terminé avant de retourner un ActionResult .

dans votre code, vous retournez sans vous assurer que DownloadAsync() a été exécuté. Cependant, vous enregistrez le résultat au membre task , de sorte qu'il est très facile de s'assurer qu'il est complet. Mettez simplement await task dans toutes vos méthodes d'action (après les avoir asynciquées) avant de retourner:

public async Task<ActionResult> IndexAsync()
{
    try
    {
        return View();
    }
    finally
    {
        await task;
    }
}

EDIT:

dans certains cas, vous pouvez avoir besoin d'appeler un async méthode que ne devrait pas compléter avant de retourner à ASP.NET . Par exemple, vous pouvez vouloir initialiser paresseusement un service d'arrière-plan tâche qui devrait continuer à fonctionner après la fin de la requête actuelle. Ce n'est pas le cas pour le code de L'OP parce que l'OP veut que la tâche soit terminée avant le retour. Cependant, si vous avez besoin de commencer et de ne pas attendre une tâche, il y a un moyen de le faire. Vous devez simplement utiliser une technique pour "échapper" à l'actuel SynchronizationContext.Current .

• ( non repris ) une caractéristique de Task.Run() est d'échapper au courant contexte de synchronisation. Cependant, les gens recommandent de ne pas utiliser ce ASP.NET parce que ASP.NET le threadpool est spécial. En outre, même à l'extérieur de ASP.NET, cette approche se traduit par un changement de contexte supplémentaire.

• ( recommandé ) un moyen sûr d'échapper au contexte de synchronisation actuel sans forcer un commutateur de contexte supplémentaire ou déranger ASP.NET 's threadpool immediately is to set SynchronizationContext.Current to null , appelez votre méthode async , puis restaurez la valeur originale .

la méthode myWebClient.DownloadStringTaskAsync fonctionne sur un thread séparé et ne bloque pas. Une solution possible est de le faire avec le Gestionnaire D'événements téléchargeable pour myWebClient et un champ de classe SemaphoreSlim.

private SemaphoreSlim signalDownloadComplete = new SemaphoreSlim(0, 1);
private bool isDownloading = false;

....

//Add to DownloadAsync() method
myWebClient.DownloadDataCompleted += (s, e) => {
 isDownloading = false;
 signalDownloadComplete.Release();
}
isDownloading = true;

...

//Add to block main calling method from returning until download is completed 
if (isDownloading)
{
   await signalDownloadComplete.WaitAsync();
}

méthode retour async Task , et ConfigureAwait(false) peut être l'un de la solution. Il agira comme async void et ne continuera pas avec le contexte sync (tant que vous ne vous souciez pas vraiment du résultat final de la méthode)

exemple de notification par courriel avec pièce jointe ..

public async Task SendNotification(string SendTo,string[] cc,string subject,string body,string path)
    {             
        SmtpClient client = new SmtpClient();
        MailMessage message = new MailMessage();
        message.To.Add(new MailAddress(SendTo));
        foreach (string ccmail in cc)
            {
                message.CC.Add(new MailAddress(ccmail));
            }
        message.Subject = subject;
        message.Body =body;
        message.Attachments.Add(new Attachment(path));
        //message.Attachments.Add(a);
        try {
             message.Priority = MailPriority.High;
            message.IsBodyHtml = true;
            await Task.Yield();
            client.Send(message);
        }
        catch(Exception ex)
        {
            ex.ToString();
        }
 }