StashApiCSharp
StashApiCSharp copied to clipboard
Basic Authentication using SecureString
I see that the StashApiCSharp supports basic authentication using string for password . wouldn't it better to add support for SecureString in order to avoid possible risk of exposing sensitive data in managed memory? I was thinking to add a new public HttpCommunicationWorker(string baseUrl, string username, SecureString password) { var httpClientHandler = new HttpClientHandler { Credentials = new NetworkCredential(username, password) };
_httpClient = new HttpClient(httpClientHandler) {BaseAddress = new Uri(baseUrl)};
}
Thanks, Frank