在 finally 块中调用 .ConfigureAwait(false) 是否安全？

Question

我有一个包装HttpClient 的“休息客户端”，其方法是异步的。 除其他原因外，我还需要使用我的休息客户端控制登录/注销过程，以免超出会话数。

其余客户端实现IDisposable，在处理客户端时，我需要检查客户端是否“仍然登录”，如果是则退出。 由于在 Dispose 方法中进行任何类型的外部调用都被认为是不好的做法，因此我有以下内容

public class MappingsController : RestController
{
    [HttpGet]
    public async Task<HttpResponseMessage> GetYears()
    {
        return await ProcessRestCall(async rc => await rc.GetYearsAsync());
    }
}

public class RestController : ApiController
{
    protected async Task<HttpResponseMessage> ProcessRestCall<T>(Func<RestClient, Task<T>> restClientCallback)
    {
        RestClient restClient = null;
        try
        {
            var credentials = GetCredentialsFromRequestHeader();
            if (credentials == null)
            {
                return Request.CreateErrorResponse(HttpStatusCode.Unauthorized, "Missing credentials from header!");
            }
            var username = credentials["Username"];
            var password = credentials["Password"];

            restClient = new RestClient(username, password);
            var authenticated = await restClient.SignInAsync();
            if (!authenticated)
            {
                return CreateErrorResponseWithRestStatus(HttpStatusCode.Unauthorized, restClient);
            }
            var result = await restClientCallback(restClient);
            // Following works, but since I need to do it in finally block in case exception happens, perhaps It should be done in finally anyways...
            //await restClient.SignOutAsync(); 
            var response = Request.CreateResponse(HttpStatusCode.OK, result);
            return response;
        }
        catch (Exception e)
        {
            return CreateErrorResponseWithRestStatus(HttpStatusCode.BadRequest, restClient, e);
        }
        finally
        {
            if (restClient != null)
            {
                if (restClient.IsSignedIn)
                {
                    //var signedOutOk = restClient.SignOutAsync();//.Result; //<-- problem - this blocks!!!
                    restClient.SignOutAsync().ConfigureAwait(false); // seems to work, but I am not sure if this is kosher + I can't get return var

                    //Logger.Warn(CultureInfo.InvariantCulture, m => m("Client was still signed in! Attempt to to sign out was {0}", signedOutOk ? "successful" : "unsuccessful"));
                }
                restClient.Dispose();
            }
        }
    }
}

Answer 1

.ConfigureAwait(false) 的使用不是问题。 你根本没有在等待任务。因为你不await它，所以await被配置做什么都没关系。

您所做的只是基本的“一劳永逸”（您可能会接受也可能不会接受）。

无论如何，您都应该删除ConfigureAwait(false)，因为它什么都不做，并且让读者感到困惑。如果您可以发送退出请求但实际上没有退出，那么这没关系。

如果您需要确保在退出请求返回之前不会调用restClient.Dispose();，那么您就有一点……问题了。问题源于注销请求可能不成功，或者更糟糕的是，它可能根本没有响应。你需要一些方法来处理它。

你不能在finally 块中使用await，但你可以或多或少地通过延续来模仿它的行为。您可能需要执行以下操作：

public static async Task DoStuff()
{
    IDisposable disposable = null;
    try { }
    finally
    {
        var task = GenerateTask();
        var continuation = Task.WhenAny(task, Task.Delay(5000))
            .ContinueWith(t =>
            {
                if (task.IsCompleted) //if false we timed out or it threw an exception
                {
                    var result = task.Result;
                    //TODO use result
                }

                disposable.Dispose();
            });
    }
}

请注意，由于您没有使用await，所以从DoStuff 返回的任务将在第一次到达finally 块时指示它“完成”；不是在延续触发并且对象被处置时。这可能会也可能不会被接受。