on.nemtina/persistence
1
0
Fork 0
Code Issues Pull Requests 3 Actions Packages Projects Releases Wiki Activity

Новая фабрика клиентов #17

Merged
on.nemtina merged 7 commits from feature/add-refit-factory into master 2025-01-10 13:54:19 +05:00
Conversation 0 Commits 7 Files Changed 33 +13 -2
2 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit d0cd647849 - Show all commits

1
DD.Persistence.Client/DD.Persistence.Client.csproj
View File

@ -53,6 +53,7 @@
<PackageReference Include="Refit" Version="8.0.0" />
<PackageReference Include="Refit.HttpClientFactory" Version="8.0.0" />
<PackageReference Include="RestSharp" Version="112.1.0" />
ng.frolov commented 2025-01-10 13:18:24 +05:00
Review
Copy Link

проверить необходимость

проверить необходимость
on.nemtina commented 2025-01-13 10:27:42 +05:00
Review
Copy Link
image
<img width="760" alt="image" src="attachments/ce817d0b-ffdb-484f-8413-72523bf0cf3a">
image.png
73 KiB
<PackageReference Include="System.Configuration.ConfigurationManager" Version="9.0.0" />
<PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="8.3.0" />
ng.frolov commented 2025-01-10 13:18:36 +05:00
Review
Copy Link

проверить необходимость

проверить необходимость
on.nemtina commented 2025-01-13 10:26:27 +05:00
Review
Copy Link

Необходимость только для этого:

Необходимость только для этого:
5252.png
22 KiB
image.png
27 KiB
5252.png image.png
</ItemGroup>

14
DD.Persistence.Client/RefitClientFactory.cs
View File

@ -1,7 +1,9 @@
using DD.Persistence.Client.Clients.Interfaces.Refit;
using DD.Persistence.Client.Helpers;
using Microsoft.Extensions.Configuration;
using Microsoft.Extensions.Logging;
using Refit;
using System.Configuration;
using System.Text.Json;
namespace DD.Persistence.Client;
rs.efremov commented 2024-12-27 14:39:26 +05:00
Review
Copy Link

А не нужно ли добавить в проект <NoWarn>$(NoWarn);1591</NoWarn>? У меня всё в варнингах почему-то

А не нужно ли добавить в проект `<NoWarn>$(NoWarn);1591</NoWarn>`? У меня всё в варнингах почему-то
👍 1
on.nemtina commented 2024-12-27 16:58:16 +05:00
Review
Copy Link

Создала отдельную задачу для этого:
https://project.ddrilling.ru/projects/persistance-service/work_packages/746/activity

Создала отдельную задачу для этого: https://project.ddrilling.ru/projects/persistance-service/work_packages/746/activity
@ -18,11 +20,19 @@ public class RefitClientFactory<T> : IRefitClientFactory<T> where T : IRefitClie
/// <summary>
///
rs.efremov commented 2024-12-27 14:28:22 +05:00
Outdated
Review
Copy Link

Пустой summary

Пустой summary
👍 1
/// </summary>
public RefitClientFactory(IConfiguration configuration)
public RefitClientFactory(IConfiguration configuration, ILogger<IRefitClientFactory<T>> logger)
{
this.client = new HttpClient();
var baseUrl = configuration.GetSection("ClientUrl").Get<string>()!;
var baseUrl = configuration.GetSection("ClientUrl").Get<string>();
if (String.IsNullOrEmpty(baseUrl))
{
var exception = new SettingsPropertyNotFoundException("В настройках конфигурации не указан адрес Persistence сервиса.");
logger.LogError(exception.Message);
throw exception;
}
client.BaseAddress = new Uri(baseUrl);
client.Authorize(configuration);
rs.efremov commented 2024-12-27 14:33:55 +05:00
Review
Copy Link

Данный метод тянет аутентификационные данные с конфигов и авторизует Http - клиента. Это нужно для тестов, но нужно ли для непосредственного использования фабрики сторонними сервисами? Если они передают уже авторизованного Http клиента - то нужно предусмотреть вызов client.Authorize(configuration); только для тестов. Например проверкой Environment, либо через конфиг

Данный метод тянет аутентификационные данные с конфигов и авторизует Http - клиента. Это нужно для тестов, но нужно ли для непосредственного использования фабрики сторонними сервисами? Если они передают уже авторизованного Http клиента - то нужно предусмотреть вызов `client.Authorize(configuration);` только для тестов. Например проверкой Environment, либо через конфиг