在 MVC 中从同步调用异步而不等待时 TPL 任务死锁

Question

我知道在同步 MVC 方法中调用异步方法时存在 TPL 死锁陷阱，同时使用 .Wait() 或 .Result 等待任务完成。

但是我们刚刚在我们的 MVC 应用程序中发现了一个奇怪的行为：同步操作调用了一个异步方法，但由于它是一个触发器，我们从未等待它完成。不过，异步方法似乎卡住了。

代码如下所示，这个奇怪的问题不是 100% 发生的。它只是在某个时候发生。

当它发生时：

1. HomeController.Index() 动作完成
2. Lo​​g.Info("Begin") 已执行。
3. SaveToDb() 完成了这项工作，但不知道它是否在完成后挂起。
4. PublishTomessageQueue() 没有完成这项工作，不知道它是否从未启动或只是卡在里面。
5. 没有调用 Log.Info("Finish")/Log.Error("Error")。

大多数时候，代码按预期工作。

ISomeInterface.Trigger() 也从其他地方调用，Windows 服务而不是 mvc，但这种奇怪的行为从未发生过。

所以我的问题是，即使 没有 .Wait() 也没有 .Result，异步任务是否有可能陷入死锁？

非常感谢。

public interface ISomeInterface
{
    Task Trigger();
}

public class SomeClass
{
    public async Task Trigger()
    {
        Log.Info("Begin");

        try
        {
            await SaveToDb();

            await PublishToMessageQueue();

            Log.Info("Finish");
        }
        catch (Exception ex)
        {
            Log.Error("Error");
        }
    }
}

public class HomeController : Controller
{
    public ISomeInterface Some { get; set; }

    public ActionResult Index()
    {

        Some.Trigger(); //<----- The thread is not blocked here.

        return View();
    }


}

Answer 1

异步方法似乎卡住了...它只是在某个时候发生...大多数时候，代码按预期工作。

是的。这段代码有几个主要问题。

首先，它可以尝试在不再存在的请求上下文上恢复。例如，Index 的请求进来了，ASP.NET 为该线程创建了一个新的请求上下文。然后它在该请求上下文中调用Index，Index 调用Some.Trigger，当Trigger 遇到它的第一个await 时，它captures that context by default 并向Index 返回一个不完整的任务。 Index 然后返回，通知 ASP.NET 请求完成； ASP.NET 发送响应，然后拆除该请求上下文。稍后，Trigger 准备在其await 之后恢复，并尝试在该请求上下文上恢复......但它不再存在（请求已经完成）。混乱随之而来。

第二个主要问题是这是"fire and forget", which is a really bad idea on ASP.NET。这是一个坏主意，因为 ASP.NET 是完全围绕请求/响应系统设计的；它具有非常有限的功能来处理请求中不存在的代码。当没有活动请求时，ASP.NET 可以（并且将）定期回收您的应用程序域和工作进程（这是必需以保持清洁）。它完全不知道您的 Trigger 代码正在运行，因为调用它的请求已经完成 - 因此，您正在运行的代码可能会定期消失。

最简单的解决方案是将此“触发”代码移动到实际请求中。比如Index可以awaitTrigger返回的任务。或者让您的页面代码向调用 Trigger（和 awaits）的 API 发出 AJAX 调用。

如果这不可行，那么我会推荐一个合适的分布式系统：让Index 将“触发请求”放入可靠队列，并由独立后端（例如 Win32 服务）处理。或者您可以使用现成的解决方案，例如 Hangfire。

Answer 2

即使没有 .Wait() 也没有 .Result，异步任务是否有可能陷入死锁？

是的，这是可能的。默认情况下，等待之后执行将被封送回原始线程。但如果线程不可用或由于某种原因被阻塞，则可能会发生死锁。

不确定这是否也是您的问题，但您可以尝试以下方法：

await SaveToDb().ConfigureAwait(false);

await PublishToMessageQueue().ConfigureAwait(false);

ConfigureAwait(false) 告诉运行器状态机可以在任何线程上继续执行。在大多数情况下，这没问题。只有在特殊情况下（例如 WinForms 或 WPF UI 线程）才需要封送回原始线程。