fix(ui): сохранять точный выбор пира при экспорте WireGuard
This commit is contained in:
1 parent
4aa4726c6b
commit
425a77a1ca
8 files changed
+68
-10
No files matched your search
@@ -59,6 +59,7 @@ import {
|
|||||||
type ServerTileItem,
|
type ServerTileItem,
|
||||||
} from "@/components/server-tile-rail"
|
} from "@/components/server-tile-rail"
|
||||||
import { toast } from "sonner"
|
import { toast } from "sonner"
|
||||||
|
import { selectWgExportPeer } from "@/lib/wg-export-peer"
|
||||||
import { ConfigHistorySheet } from "@/components/config-history-sheet"
|
import { ConfigHistorySheet } from "@/components/config-history-sheet"
|
||||||
import type { ConfigRevisionDto } from "@/lib/config-revisions"
|
import type { ConfigRevisionDto } from "@/lib/config-revisions"
|
||||||
import {
|
import {
|
||||||
@@ -601,12 +602,14 @@ function WireGuardPageContent({ selectedServerId, selectedInterfaceId, allInterf
|
|||||||
exportPending.current = epoch
|
exportPending.current = epoch
|
||||||
setExportBusy(true)
|
setExportBusy(true)
|
||||||
try {
|
try {
|
||||||
|
const peer = format === "peer-conf" ? selectWgExportPeer(exportIface.peers, exportPeerId) : undefined
|
||||||
|
if (format === "peer-conf" && !peer) throw new Error("Выбранный пир не найден. Обновите список и выберите пира заново.")
|
||||||
const res = await exportWireGuard(backendUrl, {
|
const res = await exportWireGuard(backendUrl, {
|
||||||
serverId: exportIface.serverId,
|
serverId: exportIface.serverId,
|
||||||
interfaceName: exportIface.name,
|
interfaceName: exportIface.name,
|
||||||
format,
|
format,
|
||||||
includePrivateKey: format !== "peer-conf",
|
includePrivateKey: format !== "peer-conf",
|
||||||
peerId: format === "peer-conf" ? (exportPeerId ?? undefined) : undefined,
|
peerId: peer?.rosId ?? peer?.id,
|
||||||
})
|
})
|
||||||
if (epoch !== exportEpoch.current) return
|
if (epoch !== exportEpoch.current) return
|
||||||
setLiveExport((prev) => ({
|
setLiveExport((prev) => ({
|
||||||
|
|||||||
@@ -1,6 +1,7 @@
|
|||||||
"use client"
|
"use client"
|
||||||
|
|
||||||
import { useMemo } from "react"
|
import { useMemo } from "react"
|
||||||
|
import { selectWgExportPeer } from "@/lib/wg-export-peer"
|
||||||
import type { WgIfaceWithServer } from "@/components/data-grids/wireguard-data-grid"
|
import type { WgIfaceWithServer } from "@/components/data-grids/wireguard-data-grid"
|
||||||
import {
|
import {
|
||||||
CodeExportSheet,
|
CodeExportSheet,
|
||||||
@@ -71,17 +72,16 @@ function WgExportSheet({
|
|||||||
})),
|
})),
|
||||||
}
|
}
|
||||||
|
|
||||||
const peer =
|
const peer = selectWgExportPeer(iface.peers, peerId)
|
||||||
(peerId
|
|
||||||
? iface.peers.find((p) => p.id === peerId || p.rosId === peerId)
|
|
||||||
: undefined) ?? iface.peers[0]
|
|
||||||
|
|
||||||
const localRsc = generateMikrotikRsc(base)
|
const localRsc = generateMikrotikRsc(base)
|
||||||
const localConf = generateNativeConf(base)
|
const localConf = generateNativeConf(base)
|
||||||
|
|
||||||
let peerConf = ""
|
let peerConf = ""
|
||||||
let peerEmpty: string | undefined
|
let peerEmpty: string | undefined
|
||||||
if (!iface.publicKey) {
|
if (peerId != null && !peer) {
|
||||||
|
peerEmpty = "Выбранный пир не найден. Обновите список и выберите пира заново."
|
||||||
|
} else if (!iface.publicKey) {
|
||||||
peerEmpty = "Нет public-key интерфейса для клиентского .conf"
|
peerEmpty = "Нет public-key интерфейса для клиентского .conf"
|
||||||
} else if (!peer) {
|
} else if (!peer) {
|
||||||
peerEmpty = "Нет пиров для экспорта Peer .conf"
|
peerEmpty = "Нет пиров для экспорта Peer .conf"
|
||||||
|
|||||||
+2
-2
@@ -80,7 +80,7 @@ flowchart LR
|
|||||||
|
|
||||||
## Текущая точка продолжения
|
## Текущая точка продолжения
|
||||||
|
|
||||||
По журналу этапы 0–8 завершены в границах локальной приёмки; внешние ограничения сохраняются. Этап 9 начат (блоки 9.1–9.6 — операции и защита экспорта WireGuard); этапы 10–12 не начаты. Этап 11 разделён: 11А — пользователи, контейнеры, сертификаты, резервирование; 11Б — оповещения, сбор данных, интеграции, релизы, настройки.
|
По журналу этапы 0–8 завершены в границах локальной приёмки; внешние ограничения сохраняются. Этап 9 начат (блоки 9.1–9.7 — операции и точный выбор пира WireGuard); этапы 10–12 не начаты. Этап 11 разделён: 11А — пользователи, контейнеры, сертификаты, резервирование; 11Б — оповещения, сбор данных, интеграции, релизы, настройки.
|
||||||
|
|
||||||
Подробные планы: [этап 6](ux-ui-stage-6.md), [этап 7](ux-ui-stage-7.md), [этап 8 и итоговая матрица](ux-ui-stage-8.md), [этап 9](ux-ui-stage-9.md).
|
Подробные планы: [этап 6](ux-ui-stage-6.md), [этап 7](ux-ui-stage-7.md), [этап 8 и итоговая матрица](ux-ui-stage-8.md), [этап 9](ux-ui-stage-9.md).
|
||||||
|
|
||||||
@@ -88,7 +88,7 @@ flowchart LR
|
|||||||
|
|
||||||
Ближайший оставшийся объём:
|
Ближайший оставшийся объём:
|
||||||
|
|
||||||
1. Этап 9: карта разделов и схема сохранены; удаление, импорт, создание, переключение и контекст экспорта WireGuard исправлены в 9.1–9.6. Далее — точный выбор пира в экспорте и история/восстановление; затем IPsec/GRE/VXLAN/Firewall/Filters.
|
1. Этап 9: карта разделов и схема сохранены; удаление, импорт, создание, переключение, контекст экспорта и выбор пира WireGuard исправлены в 9.1–9.7. Далее — история/восстановление; затем IPsec/GRE/VXLAN/Firewall/Filters.
|
||||||
2. Сохранять контекст устройства, введённые значения при ошибке и подтверждать результат операций по API; локальные моки проверяют клиентские сценарии, реальные сетевые операции требуют тестового окружения.
|
2. Сохранять контекст устройства, введённые значения при ошибке и подтверждать результат операций по API; локальные моки проверяют клиентские сценарии, реальные сетевые операции требуют тестового окружения.
|
||||||
3. В этапе 12 выполнить полный аудит доступности (скринридер, контраст), производительности и сквозную регрессию. Ограничения этапа 8 — удалённые серверы месячного каталога, отсутствие экспорта и автоматического переподключения SSE — остаются явно зафиксированными, без заявления о реализации.
|
3. В этапе 12 выполнить полный аудит доступности (скринридер, контраст), производительности и сквозную регрессию. Ограничения этапа 8 — удалённые серверы месячного каталога, отсутствие экспорта и автоматического переподключения SSE — остаются явно зафиксированными, без заявления о реализации.
|
||||||
|
|
||||||
|
|||||||
@@ -25,12 +25,19 @@
|
|||||||
| 6 | Навигация и дашборд | Завершён 2026-09-20 в границах локальной приёмки; ограничения ниже |
|
| 6 | Навигация и дашборд | Завершён 2026-09-20 в границах локальной приёмки; ограничения ниже |
|
||||||
| 7 | Серверы и мониторинг: детали, формы, URL-контекст, права | Завершён в границах локальной приёмки 2026-09-23; внешние ограничения ниже |
|
| 7 | Серверы и мониторинг: детали, формы, URL-контекст, права | Завершён в границах локальной приёмки 2026-09-23; внешние ограничения ниже |
|
||||||
| 8 | Трафик и статистика | Завершён 2026-09-28 в границах локальной приёмки; ограничения в блоке 8.13 |
|
| 8 | Трафик и статистика | Завершён 2026-09-28 в границах локальной приёмки; ограничения в блоке 8.13 |
|
||||||
| 9 | VPN, Firewall и Filters | В работе: блоки 9.1–9.6 — операции и защита экспорта WireGuard |
|
| 9 | VPN, Firewall и Filters | В работе: блоки 9.1–9.7 — операции и точный выбор пира WireGuard |
|
||||||
| 10 | Маршрутизация, диагностика, карта и терминал | Не начат |
|
| 10 | Маршрутизация, диагностика, карта и терминал | Не начат |
|
||||||
| 11А | Пользователи, контейнеры, сертификаты и резервные копии | Не начат |
|
| 11А | Пользователи, контейнеры, сертификаты и резервные копии | Не начат |
|
||||||
| 11Б | Оповещения, сбор данных, интеграции, релизы и настройки | Не начат |
|
| 11Б | Оповещения, сбор данных, интеграции, релизы и настройки | Не начат |
|
||||||
| 12 | Итоговая доступность, адаптивность, производительность и регрессия | Не начат |
|
| 12 | Итоговая доступность, адаптивность, производительность и регрессия | Не начат |
|
||||||
|
|
||||||
|
## Блок 9.7 — точный выбор пира для экспорта (2026-09-29)
|
||||||
|
|
||||||
|
- Единая функция выбора используется локальным предпросмотром и запросом API. Явный ID ищется по id/rosId; если не найден, другой пир не подставляется. Первый пир выбирается только при отсутствии выбора, как в backend. Локальное пустое состояние объясняет необходимость обновить список; серверный запрос при отсутствующем пире не отправляется.
|
||||||
|
- API получает rosId выбранного пира, с id как запасным вариантом. Peer .conf сохраняет includePrivateKey=false. Схемы/сетевые правила backend не менялись.
|
||||||
|
- Прошли 4 профильных E2E экспорта (включая новый сценарий второго пира с отличающимися id/rosId), 2 unit-теста выбора, lint, TypeScript и diff-check. Неизвестный/пустой ID покрыты unit-тестом; полноценный браузерный сценарий устаревшего выбора не заявляется. Production build не повторялся; последний успешный — 9.4.
|
||||||
|
- Получен ReUI checklist, композиция CodeExportSheet сохранена. Реальные RouterOS/JWT/ключи не проверены. Далее — история/восстановление WireGuard, затем остальные разделы этапа 9. По разрешению пользователя работа продолжена сверх резерва; использовано 84% пятичасового и 60% недельного лимита. Локальный коммит без push.
|
||||||
|
|
||||||
## Блок 9.6 — контекст ответа экспорта WireGuard (2026-09-29)
|
## Блок 9.6 — контекст ответа экспорта WireGuard (2026-09-29)
|
||||||
|
|
||||||
- Каждое открытие/закрытие панели меняет поколение запроса. Поздние результат, ошибка и finally старого экспорта не меняют новую панель и не снимают её busy. Размонтирование также инвалидирует ответ. Повторный запрос в текущей панели защищён синхронной блокировкой; закрытие доступно.
|
- Каждое открытие/закрытие панели меняет поколение запроса. Поздние результат, ошибка и finally старого экспорта не меняют новую панель и не снимают её busy. Размонтирование также инвалидирует ответ. Повторный запрос в текущей панели защищён синхронной блокировкой; закрытие доступно.
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
# Этап 9: VPN и сетевые операции
|
# Этап 9: VPN и сетевые операции
|
||||||
|
|
||||||
Статус: начат 2026-09-28, выполнены блоки 9.1–9.6. Основа — существующие ReUI PRO/base-nova, Frame, ServerTileRail, DataGrid и формы. Правила применения сетевой конфигурации не меняем.
|
Статус: начат 2026-09-28, выполнены блоки 9.1–9.7. Основа — существующие ReUI PRO/base-nova, Frame, ServerTileRail, DataGrid и формы. Правила применения сетевой конфигурации не меняем.
|
||||||
|
|
||||||
## Схема и референсы принятого плана
|
## Схема и референсы принятого плана
|
||||||
|
|
||||||
@@ -65,3 +65,7 @@
|
|||||||
## Блок 9.6 — контекст экспорта (2026-09-29)
|
## Блок 9.6 — контекст экспорта (2026-09-29)
|
||||||
|
|
||||||
Результат, ошибка и завершение запроса привязаны к открытию панели; закрытие/новое открытие/размонтирование инвалидируют прежний ответ. Повтор защищён замком. Схема ответа и запрошенный формат проверяются. Три новых E2E, lint и TypeScript прошли; итог общего набора — в журнале. Далее: точный выбор пира, затем история/восстановление. Реальный экспорт ключей не проверен; сетевой запрос после закрытия не отменяется.
|
Результат, ошибка и завершение запроса привязаны к открытию панели; закрытие/новое открытие/размонтирование инвалидируют прежний ответ. Повтор защищён замком. Схема ответа и запрошенный формат проверяются. Три новых E2E, lint и TypeScript прошли; итог общего набора — в журнале. Далее: точный выбор пира, затем история/восстановление. Реальный экспорт ключей не проверен; сетевой запрос после закрытия не отменяется.
|
||||||
|
|
||||||
|
## Блок 9.7 — точный пир (2026-09-29)
|
||||||
|
|
||||||
|
Явный неизвестный ID не заменяется первым пиром; локальный и серверный экспорт используют общую функцию выбора по id/rosId. В API передаётся RouterOS ID выбранного пира. Прошли 4 E2E экспорта, 2 unit-теста, lint и TypeScript; неизвестный ID проверен на уровне функции. Далее — история и восстановление WireGuard. Реальное устройство не проверено.
|
||||||
@@ -0,0 +1,16 @@
|
|||||||
|
import { test } from "node:test"
|
||||||
|
import assert from "node:assert/strict"
|
||||||
|
import { selectWgExportPeer } from "./wg-export-peer"
|
||||||
|
|
||||||
|
const peers = [{ id: "first", rosId: "*1" }, { id: "second", rosId: "*2" }]
|
||||||
|
test("peer export resolves row and RouterOS IDs without changing an explicit selection", () => {
|
||||||
|
assert.equal(selectWgExportPeer(peers, "second"), peers[1])
|
||||||
|
assert.equal(selectWgExportPeer(peers, "*2"), peers[1])
|
||||||
|
assert.equal(selectWgExportPeer(peers, "missing"), undefined)
|
||||||
|
assert.equal(selectWgExportPeer(peers, ""), undefined)
|
||||||
|
})
|
||||||
|
test("peer export defaults only when no selection was provided", () => {
|
||||||
|
assert.equal(selectWgExportPeer(peers), peers[0])
|
||||||
|
assert.equal(selectWgExportPeer(peers, null), peers[0])
|
||||||
|
assert.equal(selectWgExportPeer([]), undefined)
|
||||||
|
})
|
||||||
@@ -0,0 +1,4 @@
|
|||||||
|
/** An explicit missing selection must never fall back to another peer. */
|
||||||
|
export function selectWgExportPeer<T extends { id?: string; rosId?: string }>(peers: readonly T[], peerId?: string | null): T | undefined {
|
||||||
|
return peerId == null ? peers[0] : peers.find(peer => peer.id === peerId || peer.rosId === peerId)
|
||||||
|
}
|
||||||
@@ -415,3 +415,27 @@ test("WireGuard: экспорт отклоняет повреждённый от
|
|||||||
await dialog.getByRole("button", { name: "Подтянуть конфиг с роутера с private-key", exact: true }).click()
|
await dialog.getByRole("button", { name: "Подтянуть конфиг с роутера с private-key", exact: true }).click()
|
||||||
await expect(dialog.locator("pre")).toContainText("VALID_EXPORT")
|
await expect(dialog.locator("pre")).toContainText("VALID_EXPORT")
|
||||||
})
|
})
|
||||||
|
|
||||||
|
test("WireGuard: экспорт второго пира использует его RouterOS ID", async ({ page }) => {
|
||||||
|
await setup(page, "/wireguard?server=1")
|
||||||
|
const fixture = wgListResponseSchema.parse({ interfaces: [{ id: "iface", rosId: "*I", serverId: "1", serverName: "Источник", name: "wg-peers", publicKey: "fixture-server-key", listenPort: 13231, mtu: 1420, enabled: true, status: "up", comment: "", peers: [
|
||||||
|
{ id: "first", rosId: "*P1", publicKey: "fixture-peer-one", allowedIps: ["10.0.1.1/32"] },
|
||||||
|
{ id: "second", rosId: "*P2", publicKey: "fixture-peer-two", allowedIps: ["10.0.2.1/32"] },
|
||||||
|
] }] })
|
||||||
|
await page.route("**/api/wireguard", route => route.fulfill({ json: fixture }))
|
||||||
|
let body: Record<string, unknown> | undefined
|
||||||
|
await page.route("**/api/wireguard/export", route => {
|
||||||
|
body = route.request().postDataJSON()
|
||||||
|
return route.fulfill({ json: { format: "peer-conf", filename: "peer.conf", content: "SECOND_PEER_EXPORT" } })
|
||||||
|
})
|
||||||
|
await page.reload()
|
||||||
|
await page.getByRole("tab", { name: /Пиры/ }).click()
|
||||||
|
await expect(page.locator("tr[data-row-id]")).toHaveCount(2)
|
||||||
|
await page.locator("tr[data-row-id]").filter({ hasText: "10.0.2.1/32" }).getByRole("button", { name: "Экспорт peer .conf" }).click()
|
||||||
|
const dialog = page.getByRole("dialog", { name: "Экспорт WireGuard" })
|
||||||
|
await expect(dialog.locator("pre")).toContainText("10.0.2.1/32")
|
||||||
|
await expect(dialog.locator("pre")).not.toContainText("10.0.1.1/32")
|
||||||
|
await dialog.getByRole("button", { name: "Подтянуть конфиг с роутера с private-key" }).click()
|
||||||
|
await expect(dialog.locator("pre")).toContainText("SECOND_PEER_EXPORT")
|
||||||
|
expect(body).toMatchObject({ serverId: "1", interfaceName: "wg-peers", peerId: "*P2", format: "peer-conf", includePrivateKey: false })
|
||||||
|
})
|
||||||
Reference in new issue
Block a user