Conversation
…tead of usual ones, removed extra methods, added Redis exceptions handling, changed cors configuration, refactored ProgramProjectGeneratorService.cs - moved all the logic into new ProgramProjectCacheService.cs + small refactoring
… small refactoring
…velopment into feature/cache
github-actions
bot
added
In progress
alxmcs
left a comment
alxmcs
left a comment
There was a problem hiding this comment.
Общее замечание: запустить клинап по солюшену
Aspire/Aspire.AppHost/AppHost.cs
Outdated
|
|
||
| var cache = builder.AddRedis("programproj-cache").WithRedisInsight(containerName: "programproj-insight"); | ||
|
|
||
| var ports = builder.Configuration.GetSection("ApiGateway:Ports").Get<int[]>() ?? throw new InvalidOperationException("api gw ports r not configured"); |
There was a problem hiding this comment.
Хотелось бы более вменяемое сообщение об ошибке
Aspire/Aspire.AppHost/AppHost.cs
Outdated
| var apiGW = builder.AddProject<Projects.Service_ApiGw>("api-gw", project => { project.ExcludeLaunchProfile = true; }) | ||
| .WithHttpEndpoint(port: 5247) | ||
| .WithHttpsEndpoint(port: 7198); | ||
| foreach (var service in apiServices) apiGW.WaitFor(service); |
There was a problem hiding this comment.
Убрать в цикл выше
Короче, такая же история, как и в #38, так что можешь посмотреть подробные комментарии там
Aspire/Aspire.AppHost/AppHost.cs
Outdated
| .WithHttpsEndpoint(port: 7198); | ||
| foreach (var service in apiServices) apiGW.WaitFor(service); | ||
| builder.AddProject<Projects.Client_Wasm>("programproj-wasm") | ||
| .WithReference(apiGW) |
There was a problem hiding this comment.
Тебе WithReference() тут не поможет, потому что это серверная функциональность, а у тебя стеналон васм, который запускается в браузере
Service.ApiGw/Service.ApiGw.http
Outdated
There was a problem hiding this comment.
Какой еще weatherforecast?
There was a problem hiding this comment.
это по дефолту создалось, не заметил
There was a problem hiding this comment.
Я бы сказал, что название данного проекта нарушает шарповые конвенции именования
| { | ||
| var services = await _services(); | ||
| if (services == null || services.Count == 0) { | ||
| throw new InvalidOperationException("no downstreaam services"); |
There was a problem hiding this comment.
Если бы это был настоящий программный продукт, тебя бы распяли за такое сообщение об ошибке
| { | ||
| return new OkResponse<ServiceHostAndPort>(services[0].HostAndPort); | ||
| } | ||
| var idxx = Math.Abs(id % services.Count); |
There was a problem hiding this comment.
Тут я бы тоже сказал, что конвенции именования нарушены
Или это просто печатка, как и со downstreaam, хз
| using Ocelot.DownstreamRouteFinder.Finder; | ||
| using Ocelot.Errors; | ||
|
|
||
| namespace Service.ApiGw.balancer; |
There was a problem hiding this comment.
Неймспейс нарушает правила именования
Service.ApiGw/Program.cs
Outdated
| }); | ||
| var app = builder.Build(); | ||
|
|
||
| app.MapDefaultEndpoints(); |
There was a problem hiding this comment.
Тебе зачем эти эндпоинты кроме как в дев среде?
Service.ApiGw/Program.cs
Outdated
| using Service.ApiGw.balancer; | ||
| var builder = WebApplication.CreateBuilder(args); |
There was a problem hiding this comment.
Ну добавь ты пустую строку между юзингами и непосредственно кодом, жопа от этого ни у кого не отвалится, зато исходник будет более читаемый
2 participants
ФИО: Кадников Николай
Номер группы: 6513
Номер лабораторной: 2
Номер варианта: 42
Краткое описание предметной области: Программный проект
Краткое описание добавленных фич: Добавлены балансировщик нагрузки и реплицирование