fix(ui): подтверждать переключение адресов Firewall

This commit is contained in:
Denozordec committed 2026-09-30 09:59:36 +07:00
1 parent 131ecb8668
commit 0c460d4767
5 files changed
+80 -7

No files matched your search

+17 -4
View File
@@ -722,6 +722,7 @@ function AddressListsTab({
onEdit,
onDelete,
showServer = false,
mutationsDisabled = false,
}: {
entries: AddressListEntry[]
onAdd: () => void
@@ -730,6 +731,7 @@ function AddressListsTab({
onEdit: (entry: AddressListEntry) => void
onDelete: (entry: AddressListEntry) => void
showServer?: boolean
mutationsDisabled?: boolean
}) {
const [search, setSearch] = useState("")
const [listFilter, setListFilter] = useState("all")
@@ -816,7 +818,7 @@ function AddressListsTab({
⏱ {e.timeout}
</span>
)}
<FormToggle checked={!e.disabled} onChange={() => onToggle(e)} />
<FormToggle checked={!e.disabled} disabled={mutationsDisabled} onChange={() => onToggle(e)} />
<DropdownMenu>
<DropdownMenuTrigger render={
<Button variant="ghost" size="icon" className="size-7 shrink-0">
@@ -825,7 +827,7 @@ function AddressListsTab({
} />
<DropdownMenuContent side="bottom" align="end">
<DropdownMenuItem onClick={() => onEdit(e)}><PencilIcon className="size-4" />Редактировать</DropdownMenuItem>
<DropdownMenuItem onClick={() => onToggle(e)}>
<DropdownMenuItem disabled={mutationsDisabled} onClick={() => onToggle(e)}>
<PowerIcon className="size-4" />{e.disabled ? "Включить" : "Отключить"}
</DropdownMenuItem>
<DropdownMenuSeparator />
@@ -2313,17 +2315,27 @@ function FirewallPageInner() {
}
async function toggleAddr(e: AddressListEntry) {
if (toggleLock.current || busy || historyRestoring) return
if (isLive) {
const key = addressKey(e)
if (!key) return
toggleLock.current = true
setTogglePending(true)
setToggleError(null)
setToggleStatus(`Ожидание изменения адреса ${e.address} (${e.list})…`)
try {
await apiFetch("/api/firewall/address-lists", {
const result = await apiFetch<{ ok: boolean }>("/api/firewall/address-lists", {
method: "PATCH",
body: JSON.stringify({ ...key, disabled: !e.disabled }),
})
if (result?.ok !== true) throw new Error("Сервер не подтвердил изменение адреса")
await loadLive()
} catch (err) {
toast.error(err instanceof Error ? err.message : "Не удалось переключить запись")
setToggleError(`${e.address} (${e.list}): ${err instanceof Error ? err.message : "Не удалось переключить запись"}`)
} finally {
toggleLock.current = false
setTogglePending(false)
setToggleStatus(null)
}
return
}
@@ -2544,6 +2556,7 @@ function FirewallPageInner() {
{chainGroup === "address-lists" ? (
<AddressListsTab
entries={familyAddrLists}
mutationsDisabled={togglePending || busy || historyRestoring}
onAdd={() => openAddAddr()}
onAddToList={(list) => openAddAddr(list)}
onToggle={(e) => { void toggleAddr(e) }}
+2 -2
View File
@@ -80,7 +80,7 @@ flowchart LR
## Текущая точка продолжения
По журналу этапы 0–8 завершены в границах локальной приёмки; внешние ограничения сохраняются. Этап 9 начат (блоки 9.1–9.25 — IPsec/GRE/VXLAN и переключение Firewall); этапы 10–12 не начаты. Этап 11 разделён: 11А — пользователи, контейнеры, сертификаты, резервирование; 11Б — оповещения, сбор данных, интеграции, релизы, настройки.
По журналу этапы 0–8 завершены в границах локальной приёмки; внешние ограничения сохраняются. Этап 9 начат (блоки 9.1–9.26 — IPsec/GRE/VXLAN и переключение Firewall); этапы 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).
@@ -88,7 +88,7 @@ flowchart LR
Ближайший оставшийся объём:
1. Этап 9: карта разделов и схема сохранены; удаление, импорт, создание, переключение, контекст экспорта и выбор пира WireGuard исправлены в 9.1–9.7; в 9.8 проверено подтверждение восстановления. В 9.9 исправлены разметка и сохранение подтверждения при ошибке у всех шести потребителей. В 9.10 исправлены ошибки и поздние ответы загрузки истории WireGuard. В 9.11 подтверждение восстановления проверено у остальных пяти потребителей. В 9.12 обработаны ошибка/повтор/поздние ответы истории IPsec. В 9.13 подключены ошибка/повтор/защита поздних ответов остальных четырёх разделов и общая валидация ревизий всех шести. В 9.14 исправлены подтверждение инициализации IPsec, сохранение формы и защита повторной отправки. В 9.15 проверены создание/редактирование пользователей IPsec и сохранение формы. В 9.16 проверены получение/повторный экспорт сертификатов, поздние ответы и соответствие отображаемого пароля архиву. В 9.17 подтверждены ответы удаления четырёх типов объектов IPsec и исправлен диалог повторов. В 9.18 исправлено подтверждение сохранения peer IPsec и поведение формы. В 9.19 исправлено подтверждение удаления GRE и сохранение диалога при ошибке. В 9.20 исправлено подтверждение включения/выключения GRE и защита повторов. В 9.21 исправлены подтверждение создания/сохранения GRE и защита формы. В 9.22 исправлены контейнер ответа, первоначальные KPI и сохранение данных при ошибке VXLAN. В 9.23 backend сообщает частичные сбои VXLAN, UI сохраняет данные неответивших устройств и показывает неизвестное состояние. В 9.24 добавлены схемы payload и тесты гонок VXLAN. В 9.25 подтверждены включение/отключение правил Firewall и защита повторов. Далее — address-list, сохранение/удаление и перемещение Firewall, затем Filters; ограничения счётчика, свежести и записи VXLAN остаются открытыми; загрузка данных IPsec и общая приёмка остаются открытыми.
1. Этап 9: карта разделов и схема сохранены; удаление, импорт, создание, переключение, контекст экспорта и выбор пира WireGuard исправлены в 9.1–9.7; в 9.8 проверено подтверждение восстановления. В 9.9 исправлены разметка и сохранение подтверждения при ошибке у всех шести потребителей. В 9.10 исправлены ошибки и поздние ответы загрузки истории WireGuard. В 9.11 подтверждение восстановления проверено у остальных пяти потребителей. В 9.12 обработаны ошибка/повтор/поздние ответы истории IPsec. В 9.13 подключены ошибка/повтор/защита поздних ответов остальных четырёх разделов и общая валидация ревизий всех шести. В 9.14 исправлены подтверждение инициализации IPsec, сохранение формы и защита повторной отправки. В 9.15 проверены создание/редактирование пользователей IPsec и сохранение формы. В 9.16 проверены получение/повторный экспорт сертификатов, поздние ответы и соответствие отображаемого пароля архиву. В 9.17 подтверждены ответы удаления четырёх типов объектов IPsec и исправлен диалог повторов. В 9.18 исправлено подтверждение сохранения peer IPsec и поведение формы. В 9.19 исправлено подтверждение удаления GRE и сохранение диалога при ошибке. В 9.20 исправлено подтверждение включения/выключения GRE и защита повторов. В 9.21 исправлены подтверждение создания/сохранения GRE и защита формы. В 9.22 исправлены контейнер ответа, первоначальные KPI и сохранение данных при ошибке VXLAN. В 9.23 backend сообщает частичные сбои VXLAN, UI сохраняет данные неответивших устройств и показывает неизвестное состояние. В 9.24 добавлены схемы payload и тесты гонок VXLAN. В 9.25 подтверждены включение/отключение правил Firewall и защита повторов. В 9.26 подтверждены включение/отключение address-list с общим замком переключений. Далее — сохранение/удаление и перемещение Firewall, затем Filters; ограничения счётчика, свежести и записи VXLAN остаются открытыми; загрузка данных IPsec и общая приёмка остаются открытыми.
2. Сохранять контекст устройства, введённые значения при ошибке и подтверждать результат операций по API; локальные моки проверяют клиентские сценарии, реальные сетевые операции требуют тестового окружения.
3. В этапе 12 выполнить полный аудит доступности (скринридер, контраст), производительности и сквозную регрессию. Ограничения этапа 8 — удалённые серверы месячного каталога, отсутствие экспорта и автоматического переподключения SSE — остаются явно зафиксированными, без заявления о реализации.
+9 -1
View File
@@ -25,12 +25,20 @@
| 6 | Навигация и дашборд | Завершён 2026-09-20 в границах локальной приёмки; ограничения ниже |
| 7 | Серверы и мониторинг: детали, формы, URL-контекст, права | Завершён в границах локальной приёмки 2026-09-23; внешние ограничения ниже |
| 8 | Трафик и статистика | Завершён 2026-09-28 в границах локальной приёмки; ограничения в блоке 8.13 |
| 9 | VPN, Firewall и Filters | В работе: блоки 9.1–9.25 — IPsec/GRE/VXLAN и переключение Firewall |
| 9 | VPN, Firewall и Filters | В работе: блоки 9.1–9.26 — IPsec/GRE/VXLAN и переключение Firewall |
| 10 | Маршрутизация, диагностика, карта и терминал | Не начат |
| 11А | Пользователи, контейнеры, сертификаты и резервные копии | Не начат |
| 11Б | Оповещения, сбор данных, интеграции, релизы и настройки | Не начат |
| 12 | Итоговая доступность, адаптивность, производительность и регрессия | Не начат |
## Блок 9.26 — переключение address-list Firewall (2026-09-30)
- PATCH записи address-list требует ok=true. Переключатели и действие переключения в меню блокируются на время запроса/обновления, используется общий с правилами синхронный замок. Обработчик также учитывает занятость формы и восстановление истории.
- Status/alert содержат адрес и имя списка. При 403, пустом ответе и ok=false состояние записи сохраняется; успешный запрос обновляет список из API. Mock-режим сохраняет локальное переключение.
- Все 9 E2E Firewall прошли: два новых сценария включения/отключения address-list с двойным нажатием, задержкой, 403, пустым ответом, ok=false и успехом; прежние семь сценариев правил и таблицы. TypeScript прошёл. Новый тест lint-чистый, исходные 4 ошибки/2 предупреждения страницы не изменились относительно HEAD. Diff-check выполнен перед коммитом. Последний production build — 9.21, повтор не выполнялся.
- Далее — сохранение/удаление и подтверждение перемещения Firewall, затем Filters. Единая блокировка всех мутаций, гонки загрузки и реальная приёмка RouterOS/JWT остаются открытыми; этап 9 не завершён.
- Квота: 18% пятичасовой/3% недельной на старте, 20%/3% после проверок. Локальный коммит без push; чужие изменения сохранены.
## Блок 9.25 — переключение правил Firewall (2026-09-30)
- PATCH правила требует ok=true. Синхронный замок блокирует повтор до ответа и обновления списка. Переключатели и действие переключения в меню disabled во время запроса; перенос строк отключён. Вызов также блокируется при занятости формы/восстановлении истории.
+4
View File
@@ -141,3 +141,7 @@ API возвращает failures, одиночный endpoint — 502 при с
## Блок 9.25 — переключение Firewall (2026-09-30)
PATCH правила требует ok=true, замок предотвращает повтор до обновления таблицы; статус/ошибка содержат контекст правила. Все 7 E2E и TypeScript прошли. Grid/тест lint-чистые, исходные замечания страницы сохранены. Далее — address-list и остальные мутации Firewall, общий замок ещё не распространён на них. Реальные устройства не проверены; последний build — 9.21.
## Блок 9.26 — переключение address-list Firewall (2026-09-30)
PATCH требует ok=true, общий замок переключений предотвращает повтор до обновления. Ошибка сохраняет состояние записи и содержит адрес/список. Все 9 E2E и TypeScript прошли, новых замечаний lint нет. Далее — сохранение/удаление и перемещение Firewall; единая блокировка всех мутаций пока не внедрена. Реальные устройства не проверены; последний build — 9.21.
+48
View File
@@ -0,0 +1,48 @@
import { test, expect } from "@playwright/test"
for (const active of [true, false]) {
test(`Firewall: переключение адреса ${active} требует подтверждения`, async ({ page }) => {
let attempts = 0
let release: (() => void) | undefined
const gate = new Promise<void>(resolve => { release = resolve })
let currentEnabled = active
const entry = { id: "address-1", rosId: "*A/1", serverId: "1", serverName: "Роутер", family: "ip", list: "trusted", address: "192.0.2.10", comment: "Адрес тест", disabled: !active }
await page.route("**/health", route => route.fulfill({ json: { ok: true } }))
await page.route("**/api/**", async route => {
const path = new URL(route.request().url()).pathname
if (route.request().method() === "PATCH") {
attempts++
expect(path).toBe("/api/firewall/address-lists")
expect(route.request().postDataJSON()).toMatchObject({ serverId: "1", rosId: "*A/1", family: "ip", disabled: active })
if (attempts === 1) { await gate; return route.fulfill({ status: 403, json: { error: "Изменение запрещено тест" } }) }
if (attempts === 2) return route.fulfill({ json: {} })
if (attempts === 3) return route.fulfill({ json: { ok: false } })
currentEnabled = !active
return route.fulfill({ json: { ok: true } })
}
return route.fulfill({ json: path === "/api/auth/config" ? { required: false }
: path === "/api/firewall/all" ? { rules: [], addressLists: [{ ...entry, disabled: !currentEnabled }] }
: path === "/api/servers" ? [{ id: 1, name: "Роутер", host: "192.0.2.1", type: "exit-node", enabled: true, country: "RU", status: "online" }] : {} })
})
await page.goto("/firewall?server=1")
await page.getByRole("button", { name: /Адр. листы/ }).click()
const toggle = page.getByRole("switch")
await expect(toggle).toHaveCount(1)
await toggle.evaluate(element => { (element as HTMLElement).click(); (element as HTMLElement).click() })
await expect.poll(() => attempts).toBe(1)
await expect(toggle).toBeDisabled()
await expect(page.getByRole("status").filter({ hasText: "Ожидание изменения адреса" })).toBeVisible()
release!()
await expect(page.getByRole("alert").filter({ hasText: "Изменение запрещено тест" })).toBeVisible()
for (let i = 2; i <= 3; i++) {
await toggle.click()
await expect.poll(() => attempts).toBe(i)
await expect(page.getByRole("alert").filter({ hasText: "Сервер не подтвердил изменение адреса" })).toBeVisible()
await expect(toggle).toHaveAttribute("aria-checked", String(active))
}
await toggle.click()
await expect(toggle).toHaveAttribute("aria-checked", String(!active))
expect(attempts).toBe(4)
await expect(page.getByRole("alert").filter({ hasText: "Сервер не подтвердил" })).toHaveCount(0)
})
}