-
Notifications
You must be signed in to change notification settings - Fork 45
Open
Description
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
Metadata
Metadata
Assignees
Labels
No labels