akamsteeg / AtleX.HaveIBeenPwned

A fully async .NET Standard client library for the API of HaveIBeenPwned.com
https://www.nuget.org/packages/AtleX.HaveIBeenPwned/
MIT License
5 stars 0 forks source link

Mark HaveIBeenPwnedClientSettings.TimeOut as obsolete #37

Closed akamsteeg closed 4 years ago

akamsteeg commented 4 years ago

In HaveIBeenPwnedClientSettings there's a TimeOut property:

https://github.com/akamsteeg/AtleX.HaveIBeenPwned/blob/a00a6ce3bd98e3d95d1d445bfa83ede97710c976/src/AtleX.HaveIBeenPwned/HaveIBeenPwnedClientSettings.cs#L26

This is used in HaveIBeenPwnedClient to set a timeout on the HttpClient that gets injected or instantiated by HaveIBeenPwnedClient itself.

This is a bit of a confusing API for several reasons:

I suggest marking HaveIBeenPwnedClientSettings.TimeOut as with an ObsoleteAttribute and ignore any value assigned to it from now on.