HttpHaveIBeenPwnedClient.GetBreachesInternalAsync(string, BreachMode, CancellationToken) returns null instead of an empty enumerable when no data was loaded #23
HttpHaveIBeenPwnedClient.GetBreachesInternalAsync(string, BreachMode, CancellationToken) returns null instead of an empty enumerable when no data was loaded. That leads to bugs on the call site because people are expecting to be able to use LINQ or something like that to search in the returned collection. Also, GetPastesInternalAsync(string, CancellationToken) does correctly return an empty enumerable so we're inconsistent now.
HttpHaveIBeenPwnedClient.GetBreachesInternalAsync(string, BreachMode, CancellationToken)
returns null instead of an empty enumerable when no data was loaded. That leads to bugs on the call site because people are expecting to be able to use LINQ or something like that to search in the returned collection. Also,GetPastesInternalAsync(string, CancellationToken)
does correctly return an empty enumerable so we're inconsistent now.