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

Merged
on.nemtina merged 7 commits from feature/add-refit-factory into master 2025-01-10 13:54:19 +05:00
Owner
No description provided.
on.nemtina added 4 commits 2024-12-27 13:36:20 +05:00
on.nemtina requested review from ng.frolov 2024-12-27 13:36:29 +05:00
ng.frolov was assigned by on.nemtina 2024-12-27 13:36:33 +05:00
on.nemtina self-assigned this 2024-12-27 13:36:33 +05:00
on.nemtina added 1 commit 2024-12-27 13:37:49 +05:00
Корректный файл appsettings.Test.json
All checks were successful
Unit tests / test (push) Successful in 1m11s
4cae12ddac
on.nemtina requested review from rs.efremov 2024-12-27 13:44:43 +05:00
rs.efremov was assigned by on.nemtina 2024-12-27 13:44:46 +05:00
rs.efremov reviewed 2024-12-27 14:28:22 +05:00
@ -0,0 +18,4 @@
private RefitSettings refitSettings;
/// <summary>
///
Collaborator

Пустой summary

Пустой summary
rs.efremov approved these changes 2024-12-27 14:44:49 +05:00
@ -0,0 +1,21 @@
using DD.Persistence.Client.Clients.Interfaces.Refit;
using System;
Collaborator

Лишние namespace-ы

Лишние namespace-ы
@ -0,0 +6,4 @@
using System.Configuration;
using System.Text.Json;
namespace DD.Persistence.Client;
Collaborator

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

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

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

Создала отдельную задачу для этого: https://project.ddrilling.ru/projects/persistance-service/work_packages/746/activity
@ -0,0 +35,4 @@
throw exception;
}
client.BaseAddress = new Uri(baseUrl);
client.Authorize(configuration);
Collaborator

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

Данный метод тянет аутентификационные данные с конфигов и авторизует Http - клиента. Это нужно для тестов, но нужно ли для непосредственного использования фабрики сторонними сервисами? Если они передают уже авторизованного Http клиента - то нужно предусмотреть вызов `client.Authorize(configuration);` только для тестов. Например проверкой Environment, либо через конфиг
on.nemtina added 1 commit 2024-12-28 15:30:42 +05:00
ng.frolov approved these changes 2025-01-10 13:34:07 +05:00
@ -23,6 +23,7 @@
</ItemGroup>
<ItemGroup>
<ProjectReference Include="..\DD.Persistence.Client\DD.Persistence.Client.csproj" />
Collaborator

убрать

убрать
@ -28,3 +29,2 @@
services.AddServices();
//DependencyInjection.MapsterSetup();
services.AddPersistenceClients();
Collaborator

убрать

убрать
@ -23,1 +22,3 @@
}
"http://schemas.xmlsoap.org/ws/2005/05/identity /claims/nameidentifier": "7d9f3574-6574-4ca3-845a-0276eb4aa8f6"
},
"ClientUrl": "http://localhost:5000/"
Collaborator

убрать

убрать
@ -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" />
Collaborator

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

проверить необходимость
Author
Owner
image
<img width="760" alt="image" src="attachments/ce817d0b-ffdb-484f-8413-72523bf0cf3a">
@ -54,3 +54,4 @@
<PackageReference Include="Refit.HttpClientFactory" Version="8.0.0" />
<PackageReference Include="RestSharp" Version="112.1.0" />
<PackageReference Include="System.Configuration.ConfigurationManager" Version="9.0.0" />
<PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="8.3.0" />
Collaborator

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

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

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

Необходимость только для этого:
@ -15,2 +20,3 @@
{
return factory.CreateClient();
var client = factory.CreateClient();
client.Authorize(configuration);
Collaborator

Авторизация должна добавляться клиенту в фабрике. У клиента не должно быть метода ауторайз

Авторизация должна добавляться клиенту в фабрике. У клиента не должно быть метода ауторайз
on.nemtina added 1 commit 2025-01-10 13:44:44 +05:00
Удаление ссылки на Clients внутри проекта с API
All checks were successful
Unit tests / test (push) Successful in 1m0s
4c2b91cd2d
on.nemtina merged commit 56a6bd0a67 into master 2025-01-10 13:54:19 +05:00
on.nemtina deleted branch feature/add-refit-factory 2025-01-10 13:54:19 +05:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: on.nemtina/persistence#17
No description provided.