babelshift / SteamWebAPI2

🎮 C# / .NET library that makes it easy to use the Steam Web API. It conveniently wraps around all of the JSON data and ugly API details with clean methods, structures and classes.
MIT License
263 stars 43 forks source link

HttpClients not being handled correctly #72

Closed dukeofsussex closed 5 years ago

dukeofsussex commented 5 years ago

Great library, but diving into the code I found a lot of this:

HttpClient httpClient = new HttpClient();
// do something
return response;

The client isn't being disposed off, kept as a singleton or injected using the factory pattern.

I have no clue whether this causes any issues though, just a heads up.

babelshift commented 5 years ago

Fixed with #73