fix(caddy): исключить _not-found из фильтра чанков закрытого контура
All checks were successful
CI Trade-In / changes (pull_request) Successful in 13s
CI / changes (pull_request) Successful in 16s
CI Trade-In / backend-tests (pull_request) Has been skipped
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / backend-tests (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
All checks were successful
CI Trade-In / changes (pull_request) Successful in 13s
CI / changes (pull_request) Successful in 16s
CI Trade-In / backend-tests (pull_request) Has been skipped
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / backend-tests (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
Ревью #3340: `chunks/app/_not-found/` — внутренняя страница Next, а не маршрут продукта; фильтр её глушил, и клиентская навигация по битому href падала в error-boundary вместо честного «не найдено». Закрытого кода она не несёт. Заодно в комментарии матчера явно зафиксировано, что `static/css/app/<маршрут>/*` оставлен без фильтра сознательно — тот же класс «имена, а не код», потолок уже задекларирован.
This commit is contained in:
parent
177207223b
commit
5d9ff13dae
1 changed files with 10 additions and 3 deletions
|
|
@ -496,10 +496,17 @@ meraocenka.ru {
|
|||
# ручек. Матчер отсекает ровно «есть подкаталог, и он не mera-public» —
|
||||
# `not` вместо негативного lookahead, RE2 его не поддерживает.
|
||||
#
|
||||
# `_not-found` в исключениях — это не маршрут продукта, а внутренняя
|
||||
# страница Next (404 при клиентской навигации по битому href). Закрытого
|
||||
# кода не несёт, а без исключения клиентский переход падал бы в
|
||||
# error-boundary вместо честного «не найдено».
|
||||
#
|
||||
# Ограничение честно: это убирает КОД, но не имена файлов — карта чанков
|
||||
# лежит в `webpack-*.js`, который лэндингу нужен, поэтому перечень
|
||||
# маршрутов из неё по-прежнему вычитывается. Полное разделение — split
|
||||
# build (отдельный public-бандл), отдельная задача.
|
||||
# маршрутов из неё по-прежнему вычитывается. По той же причине оставлены
|
||||
# без фильтра и `static/css/app/<маршрут>/*`: это тот же класс «имена, а не
|
||||
# код», и городить второй матчер ради него смысла нет. Полное разделение —
|
||||
# split build (отдельный public-бандл), отдельная задача.
|
||||
#
|
||||
# `respond` внутри `handle` выполняется раньше `reverse_proxy` (порядок
|
||||
# директив Caddy), поэтому вложенный матчер не зависит от сортировки
|
||||
|
|
@ -507,7 +514,7 @@ meraocenka.ru {
|
|||
handle /trade-in/_next/static/* {
|
||||
@foreignRouteChunk {
|
||||
path_regexp ^/trade-in/_next/static/chunks/app/[^/]+/
|
||||
not path /trade-in/_next/static/chunks/app/mera-public/*
|
||||
not path /trade-in/_next/static/chunks/app/mera-public/* /trade-in/_next/static/chunks/app/_not-found/*
|
||||
}
|
||||
respond @foreignRouteChunk 404
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue