Update WebPushClient.cs

[fnoring]

Allow for disposing of the internally created HttpClient

[tuwrraphael]

Good suggestion, but its required to set the _httpClient to null at least. I linked the Disposable implementation suggestion from Microsoft in the comment.

[tuwrraphael]

Dispose should be implemented like that (as described in https://docs.microsoft.com/en-us/visualstudio/code-quality/ca1063-implement-idisposable-correctly?view=vs-2017)

public class WebPushClient : IDisposable { public void Dispose() { Dispose(true); } public override void Dispose(bool disposing) { if (disposing) { if (_httpClient != null) { _httpClient.Dispose(); _httpClient = null; } } } }

[tuwrraphael]

Personally, I prefer generating the HttpRequestMessage using this class and feeding it to a HttClient form outside. On this way IHttpClientFactory etc can be used.

[coryjthompson]

Handled in 80f8d43. Thank you.