Async version of Context.ReturnFile?

[GrabzIt]

Love Dotvvm! Maybe there is a reason not to do it. However, as ReturnFile is doing disk I/O it should probably be async. Or at least have an async version. Looking, at the code it seems to only be a minor change.

dotvvm/src/Framework/Framework/Hosting/DotvvmRequestContextExtensions.cs

Line 218 in 4d1def0

public static void ReturnFile(this IDotvvmRequestContext context, Stream stream, string fileName, string mimeType, IEnumerable<KeyValuePair<string, string>>? additionalHeaders = null, string? attachmentDispositionType = null)

[exyi]

You are right. We'll need to add an async version (and probably make this one deprecated)

[tomasherceg]

Wow, we are calling .Result inside because the underlying service is already async. It seems we forgot to update this method. :-(