🔥 Хитрая .NET-задача

Дан код:

public class Fetcher
{
    private readonly HttpClient _client = new HttpClient();

    public async Task<string> DownloadAsync(string url)
    {
        await Task.Delay(10);
        return await _client.GetStringAsync(url);
    }
}

public async Task RunAsync()
{
    var fetcher = new Fetcher();
    var tasks = Enumerable.Range(0, 5)
                          .Select(_ => fetcher.DownloadAsync("https://example.com"))
                          .ToList();

    await Task.WhenAll(tasks);

    Console.WriteLine("Done");
}

Вопросы:

1) Код может «подвиснуть» или упасть в продакшене.
Какие два скрытых дефекта здесь спрятаны?

2) Как их исправить без изменения сигнатур методов?

3) Почему HttpClient здесь одновременно правильно и неправильно?

4) Что произойдет, если:

заменить HttpClient на HttpClientFactory;

- убрать Task.Delay(10)?

5) Как переписать класс так, чтобы он был:

- потокобезопасным
- быстрым
- без скрытных deadlock-сценариев
- и выдерживал сотни запросов в секунду?

Требования:

- нельзя менять логику RunAsync
- нельзя кэшировать результат
- код должен оставаться асинхронным