Using a proxy for tonClient #121
Conversation
Changing the data type of an Ip address variable; Adding a value constraint for the port variable Adding a description to the fields of the Proxy class
| ProxyType.HTTP => "http://" + httpApiParameters.Proxy.Ip + ":" + httpApiParameters.Proxy.Port, | ||
| ProxyType.HTTPS => "https://" + httpApiParameters.Proxy.Ip + ":" + httpApiParameters.Proxy.Port, | ||
| ProxyType.Socks4 => "socks4://" + httpApiParameters.Proxy.Ip + ":" + httpApiParameters.Proxy.Port, | ||
| ProxyType.Socks5 => "socks5://" + httpApiParameters.Proxy.Ip + ":" + httpApiParameters.Proxy.Port, | ||
| _ => throw new ArgumentOutOfRangeException() |
There was a problem hiding this comment.
Как будто бы можно объединить повторяющийся код "protocol://" + httpApiParameters.Proxy.Ip + ":" + httpApiParameters.Proxy.Port.
Мб UriBuilder
There was a problem hiding this comment.
Partial, в данном случае, выглядит удобнее всего. В Models.cs у нас указываются только поля и их типы в разных классах. Реализация работы методов, на мой взгляд, там не к месту.
На счет логики сравнения прокси. А как по-другому указывать тип прокси? Как-то из enum доставать? Я просто не знаю)
| if (httpApiParameters.Proxy != null) | ||
| { | ||
| WebProxy webProxy = new WebProxy | ||
| { | ||
| Address = new Uri( httpApiParameters.Proxy.ProxyType switch | ||
| { | ||
| ProxyType.HTTP => "http://" + httpApiParameters.Proxy.Ip + ":" + httpApiParameters.Proxy.Port, | ||
| ProxyType.HTTPS => "https://" + httpApiParameters.Proxy.Ip + ":" + httpApiParameters.Proxy.Port, | ||
| ProxyType.Socks4 => "socks4://" + httpApiParameters.Proxy.Ip + ":" + httpApiParameters.Proxy.Port, | ||
| ProxyType.Socks5 => "socks5://" + httpApiParameters.Proxy.Ip + ":" + httpApiParameters.Proxy.Port, | ||
| _ => throw new ArgumentOutOfRangeException() | ||
| }), | ||
| Credentials = new NetworkCredential( | ||
| userName: httpApiParameters.Proxy.UserName, | ||
| password: httpApiParameters.Proxy.Password | ||
| ) | ||
| }; | ||
| HttpClientHandler httpClientHandler = new HttpClientHandler | ||
| { | ||
| Proxy = webProxy | ||
| }; |
There was a problem hiding this comment.
Можно вынести в приватный метод по типу:
private HttpClient CreateHttpClient(HttpApiParameters httpApiParameters)и там можно будет повысить читаемость кода (понизим вложенность и облегчим чуть конструктор)
if (httpApiParameters.Proxy == null)
{
return new HttpClient();
}
var proxyAddress = BuildProxyAddress(httpApiParameters.Proxy);
var webProxy = new WebProxy {};
var httpClientHandler = new HttpClientHandler
{
Proxy = webProxy
};
return new HttpClient(httpClientHandler);+мб можно вынести и переиспользовать для других типов хттп клиентов
There was a problem hiding this comment.
Кстати да, факты, так было бы удобней
I suggest adding the ability to use a proxy for tonClient