From 0adb6d0e580fb9899d09e543e44a1723f4f37690 Mon Sep 17 00:00:00 2001
From: "gorbenko.pavel" <gorbenko.pavel@tansu.kz>
Date: Mon, 24 Aug 2026 17:02:30 +0500
Subject: [PATCH] =?UTF-8?q?=D0=A3=D0=B1=D1=80=D0=B0=D0=BD=D1=8B=20=D0=BF?=
 =?UTF-8?q?=D1=80=D0=B5=D0=B4=D1=83=D0=BF=D1=80=D0=B5=D0=B6=D0=B4=D0=B5?=
 =?UTF-8?q?=D0=BD=D0=B8=D1=8F=20=D1=81=D0=B1=D0=BE=D1=80=D0=BA=D0=B8=20?=
 =?UTF-8?q?=D0=B1=D1=8D=D0=BA=D0=B5=D0=BD=D0=B4=D0=B0?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

NU1903: транзитивная Microsoft.OpenApi 2.0.0 (GHSA-v5pm-xwqc-g5wc, high)
запиннена на 2.7.5 — первая исправленная версия; работает через уже
включённый CentralPackageTransitivePinningEnabled, как и Newtonsoft.Json.

NU1510: убран PackageReference Microsoft.Extensions.Http из Infrastructure —
пакет приходит из общего фреймворка по FrameworkReference
Microsoft.AspNetCore.App, AddHttpClient продолжает работать.

ASPDEPR005: KnownNetworks -> KnownIPNetworks.

CS8604 (x5): Warning имеет тип string?, и Shouldly не сужает его после
ShouldNotBeNull — значение сохраняется в локальную переменную.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 backend/Directory.Packages.props              |  7 +++++-
 backend/Tnsu.ApprovalRoute.Api/Program.cs     |  2 +-
 .../Tnsu.ApprovalRoute.Infrastructure.csproj  |  1 -
 .../TemplateRouteServiceTests.cs              | 25 +++++++++++--------
 4 files changed, 22 insertions(+), 13 deletions(-)

diff --git a/backend/Directory.Packages.props b/backend/Directory.Packages.props
index 52a390c..df56e95 100644
--- a/backend/Directory.Packages.props
+++ b/backend/Directory.Packages.props
@@ -25,7 +25,12 @@
     <PackageVersion Include="Microsoft.EntityFrameworkCore.Relational" Version="10.0.10" />
     <PackageVersion Include="Microsoft.Extensions.Configuration" Version="10.0.10" />
     <PackageVersion Include="Microsoft.Extensions.DependencyInjection" Version="10.0.10" />
-    <PackageVersion Include="Microsoft.Extensions.Http" Version="10.0.10" />
+    <!--
+      Только чтобы поднять транзитивную версию из Microsoft.AspNetCore.OpenApi (2.0.0
+      с уязвимостью GHSA-v5pm-xwqc-g5wc, исправлена в 2.7.5) — напрямую не используется.
+      Убрать, когда Microsoft.AspNetCore.OpenApi сам подтянет исправленную версию.
+    -->
+    <PackageVersion Include="Microsoft.OpenApi" Version="2.7.5" />
     <PackageVersion Include="Moq" Version="4.20.72" />
     <!-- Только чтобы поднять транзитивную версию из Hangfire.Core — напрямую не используется. -->
     <PackageVersion Include="Newtonsoft.Json" Version="13.0.4" />
diff --git a/backend/Tnsu.ApprovalRoute.Api/Program.cs b/backend/Tnsu.ApprovalRoute.Api/Program.cs
index 2390684..16109f4 100644
--- a/backend/Tnsu.ApprovalRoute.Api/Program.cs
+++ b/backend/Tnsu.ApprovalRoute.Api/Program.cs
@@ -40,7 +40,7 @@ builder.Services.Configure<ForwardedHeadersOptions>(options =>
     options.ForwardedHeaders =
         ForwardedHeaders.XForwardedFor | ForwardedHeaders.XForwardedProto;
 
-    options.KnownNetworks.Clear();
+    options.KnownIPNetworks.Clear();
     options.KnownProxies.Clear();
 });
 
diff --git a/backend/Tnsu.ApprovalRoute.Infrastructure/Tnsu.ApprovalRoute.Infrastructure.csproj b/backend/Tnsu.ApprovalRoute.Infrastructure/Tnsu.ApprovalRoute.Infrastructure.csproj
index 7d5e20c..7a0e4c8 100644
--- a/backend/Tnsu.ApprovalRoute.Infrastructure/Tnsu.ApprovalRoute.Infrastructure.csproj
+++ b/backend/Tnsu.ApprovalRoute.Infrastructure/Tnsu.ApprovalRoute.Infrastructure.csproj
@@ -13,7 +13,6 @@
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>
     <PackageReference Include="Microsoft.EntityFrameworkCore.Relational" />
-    <PackageReference Include="Microsoft.Extensions.Http" />
     <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" />
     <!-- Даёт доступ к IHttpContextAccessor из общего фреймворка ASP.NET Core (тот же тип, что в Api). -->
     <FrameworkReference Include="Microsoft.AspNetCore.App" />
diff --git a/backend/tests/Tnsu.ApprovalRoute.IntegrationTests/ApplicationServices/TemplateRouteServiceTests.cs b/backend/tests/Tnsu.ApprovalRoute.IntegrationTests/ApplicationServices/TemplateRouteServiceTests.cs
index 214c438..d3167d6 100644
--- a/backend/tests/Tnsu.ApprovalRoute.IntegrationTests/ApplicationServices/TemplateRouteServiceTests.cs
+++ b/backend/tests/Tnsu.ApprovalRoute.IntegrationTests/ApplicationServices/TemplateRouteServiceTests.cs
@@ -332,8 +332,9 @@ public sealed class TemplateRouteServiceTests : IClassFixture<PostgresDatabaseFi
         resolved.IsSuccess.ShouldBeTrue();
         resolved.Value.Steps.ShouldHaveSingleItem();
         resolved.Value.Steps[0].ExecutorEmail.ShouldBeNull();
-        resolved.Value.Steps[0].Warning.ShouldNotBeNull();
-        resolved.Value.Steps[0].Warning.ShouldContain("departmentId");
+        var warning = resolved.Value.Steps[0].Warning;
+        warning.ShouldNotBeNull();
+        warning.ShouldContain("departmentId");
     }
 
     [Fact(DisplayName = "Резолв маршрута с шагом «Руководитель указанного отдела» и переданным departmentId — резолвится через портал")]
@@ -413,8 +414,9 @@ public sealed class TemplateRouteServiceTests : IClassFixture<PostgresDatabaseFi
         resolved.IsSuccess.ShouldBeTrue();
         resolved.Value.Steps.ShouldHaveSingleItem();
         resolved.Value.Steps[0].ExecutorEmail.ShouldBeNull();
-        resolved.Value.Steps[0].Warning.ShouldNotBeNull();
-        resolved.Value.Steps[0].Warning.ShouldContain(targetDepartmentId.ToString());
+        var warning = resolved.Value.Steps[0].Warning;
+        warning.ShouldNotBeNull();
+        warning.ShouldContain(targetDepartmentId.ToString());
     }
 
     [Fact(DisplayName = "Резолв маршрута включает шаг с условием «больше», когда переданное значение поля его удовлетворяет")]
@@ -723,8 +725,9 @@ public sealed class TemplateRouteServiceTests : IClassFixture<PostgresDatabaseFi
         resolved.IsSuccess.ShouldBeTrue();
         resolved.Value.Steps.ShouldHaveSingleItem();
         resolved.Value.Steps[0].ExecutorEmail.ShouldBeNull();
-        resolved.Value.Steps[0].Warning.ShouldNotBeNull();
-        resolved.Value.Steps[0].Warning.ShouldContain("руководитель");
+        var warning = resolved.Value.Steps[0].Warning;
+        warning.ShouldNotBeNull();
+        warning.ShouldContain("руководитель");
     }
 
     [Fact(DisplayName = "Резолв шага с должностью возвращает warning, если сотрудник по должности не найден")]
@@ -751,8 +754,9 @@ public sealed class TemplateRouteServiceTests : IClassFixture<PostgresDatabaseFi
         resolved.IsSuccess.ShouldBeTrue();
         resolved.Value.Steps.ShouldHaveSingleItem();
         resolved.Value.Steps[0].ExecutorEmail.ShouldBeNull();
-        resolved.Value.Steps[0].Warning.ShouldNotBeNull();
-        resolved.Value.Steps[0].Warning.ShouldContain("должности");
+        var warning = resolved.Value.Steps[0].Warning;
+        warning.ShouldNotBeNull();
+        warning.ShouldContain("должности");
     }
 
     [Fact(DisplayName = "Резолв шага с ролью в проекте возвращает warning, если исполнитель с такой ролью не найден")]
@@ -779,8 +783,9 @@ public sealed class TemplateRouteServiceTests : IClassFixture<PostgresDatabaseFi
         resolved.IsSuccess.ShouldBeTrue();
         resolved.Value.Steps.ShouldHaveSingleItem();
         resolved.Value.Steps[0].ExecutorEmail.ShouldBeNull();
-        resolved.Value.Steps[0].Warning.ShouldNotBeNull();
-        resolved.Value.Steps[0].Warning.ShouldContain("роль");
+        var warning = resolved.Value.Steps[0].Warning;
+        warning.ShouldNotBeNull();
+        warning.ShouldContain("роль");
     }
 
     [Fact(DisplayName = "Резолв маршрута откатывается на маршрут без отдела, если точного совпадения по отделу нет")]