Skip to content

Карпов Владимир Лаб. 2 Группа 6512#106

Open
Vouchiko wants to merge 10 commits intoitsecd:mainfrom
Vouchiko:main
Open

Карпов Владимир Лаб. 2 Группа 6512#106
Vouchiko wants to merge 10 commits intoitsecd:mainfrom
Vouchiko:main

Conversation

@Vouchiko
Copy link
Copy Markdown

ФИО: Карпов Владимир
Номер группы: 6512
Номер лабораторной: 2
Номер варианта: 23
Краткое описание предметной области: Товар на складе
Краткое описание добавленных фич: Реализован query based балансировщик нагрузки и интегрирован в Ocelot для распределения входящих запросов между пятью экземплярами

@github-actions github-actions bot added In progress Код в процессе проверки Lab 2 Лабораторная №2. Балансировка нагрузки labels Apr 14, 2026
@github-actions github-actions bot requested a review from Gwymlas April 14, 2026 06:36
@Vouchiko
Copy link
Copy Markdown
Author

photo_1_2026-04-14_10-39-12
photo_2_2026-04-14_10-39-12
photo_3_2026-04-14_10-39-12


public async Task<Response<ServiceHostAndPort>> LeaseAsync(HttpContext httpContext)
{
var services = await servicesProvider();
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Возможно стоит добавить проверку, что services.Count не равно нулю

Comment on lines +26 to +28



Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лишние строки

Comment on lines +2 to +9
"Logging": {
"LogLevel": {
"Default": "Information",
"Microsoft.AspNetCore": "Warning"
}
},
"AllowedHosts": "*",
"BaseAddress": "http://localhost:5300/warehouse-item"
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Раньше отступы лучше выглядели

builder.Services.AddCors(static options =>
{
options.AddPolicy(CorsPolicyName, static policy =>
policy.AllowAnyOrigin()
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно еще добавить разрешенные origins
Также удалить cors из сервиса генерации, поскольку клиент теперь обращается к апи гейтвею

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

In progress Код в процессе проверки Lab 2 Лабораторная №2. Балансировка нагрузки

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants