Je suis d'accord avec la réponse de TheWhiteRabbit mais si vous avez beaucoup d'appels en utilisant HttpClient, le code semble un peu répétitif à mon avis.
Je pense qu'il y a 2 façons d'améliorer un peu la réponse.
Créez une classe d'assistance pour créer le client:
public static class ClientHelper
{
// Basic auth
public static HttpClient GetClient(string username,string password)
{
var authValue = new AuthenticationHeaderValue("Basic", Convert.ToBase64String(Encoding.UTF8.GetBytes($"{username}:{password}")));
var client = new HttpClient(){
DefaultRequestHeaders = { Authorization = authValue}
//Set some other client defaults like timeout / BaseAddress
};
return client;
}
// Auth with bearer token
public static HttpClient GetClient(string token)
{
var authValue = new AuthenticationHeaderValue("Bearer", token);
var client = new HttpClient(){
DefaultRequestHeaders = { Authorization = authValue}
//Set some other client defaults like timeout / BaseAddress
};
return client;
}
}
Usage:
using(var client = ClientHelper.GetClient(username,password))
{
//Perform some http call
}
using(var client = ClientHelper.GetClient(token))
{
//Perform some http call
}
Créez une méthode d'extension:
Ne gagne pas de prix de beauté mais fonctionne très bien :)
public static class HttpClientExtentions
{
public static AuthenticationHeaderValue ToAuthHeaderValue(this string username, string password)
{
return new AuthenticationHeaderValue("Basic",
Convert.ToBase64String(
System.Text.Encoding.ASCII.GetBytes(
$"{username}:{password}")));
}
}
Usage:
using (var client = new HttpClient())
{
client.DefaultRequestHeaders.Authorization = _username.ToAuthHeaderValue(_password);
}
Encore une fois, je pense que 2 options ci-dessus rendent le client utilisant l'instruction un peu moins répétitif. Gardez à l'esprit qu'il est préférable de réutiliser le HttpClient si vous effectuez plusieurs appels http, mais je pense que c'est un peu hors de portée pour cette question.