chore(preview): remove basic auth от /preview/* #270

Merged

lekss361 merged 1 commit from chore/preview-remove-auth into main

2026-05-17 08:39:29 +00:00

lekss361 commented

2026-05-17 08:38:11 +00:00

Owner

Убран basicauth блок из Caddyfile для handle_path /preview/*. Публичный доступ по запросу 2026-05-17.

После merge + deploy:

https://gendsgn.ru/preview/landing.html — открывается без логина
https://gendsgn.ru/preview/ — Caddy browse index

Search engines потенциально могут проиндексировать. Если нужна защита от индексации — добавить robots.txt отдельно.

Убран `basicauth` блок из `Caddyfile` для `handle_path /preview/*`. Публичный доступ по запросу 2026-05-17. После merge + deploy: - `https://gendsgn.ru/preview/landing.html` — открывается без логина - `https://gendsgn.ru/preview/` — Caddy browse index Search engines потенциально могут проиндексировать. Если нужна защита от индексации — добавить `robots.txt` отдельно.

lekss361 added 1 commit 2026-05-17 08:38:11 +00:00

chore(preview): remove basic auth от /preview/* — публичный доступ 7a2ba4d35a

lekss361 merged commit a4247e1d96 into main

2026-05-17 08:39:29 +00:00

lekss361 referenced this pull request from a commit

2026-05-17 08:39:31 +00:00

Merge pull request 'chore(preview): remove basic auth от /preview/*' (#270) from chore/preview-remove-auth into main

lekss361 commented

2026-05-17 08:39:47 +00:00

Author

Owner

Merged via deep-code-reviewer — verdict APPROVE.

Diff проверен:

Удалён только basicauth { ... } block, handle_path /preview/*, root * /srv/preview, file_server browse сохранены
Caddyfile syntax валиден после удаления
Другие routes (/, /api/*) не затронуты
Public directory listing — intended per PR description

Smoke после деплоя:

curl -i https://gendsgn.ru/preview/landing.html   # ожидаем 200, без 401
curl -i https://gendsgn.ru/preview/                # ожидаем 200 + Caddy browse index

Follow-up (отдельным chore, recommended):

robots.txt с Disallow: /preview/* чтобы исключить индексацию search engines
Альтернатива: <meta name="robots" content="noindex"> в HTML mockups

Note: bcrypt hash был публично в репо — теперь удалён, но всё ещё доступен в git history. Если когда-то понадобится восстановить basic auth — генерировать новый hash.

Merged via deep-code-reviewer — verdict APPROVE. **Diff проверен:** - Удалён только `basicauth { ... }` block, `handle_path /preview/*`, `root * /srv/preview`, `file_server browse` сохранены - Caddyfile syntax валиден после удаления - Другие routes (`/`, `/api/*`) не затронуты - Public directory listing — intended per PR description **Smoke после деплоя:** ``` curl -i https://gendsgn.ru/preview/landing.html # ожидаем 200, без 401 curl -i https://gendsgn.ru/preview/ # ожидаем 200 + Caddy browse index ``` **Follow-up (отдельным chore, recommended):** - `robots.txt` с `Disallow: /preview/*` чтобы исключить индексацию search engines - Альтернатива: `<meta name="robots" content="noindex">` в HTML mockups Note: bcrypt hash был публично в репо — теперь удалён, но всё ещё доступен в git history. Если когда-то понадобится восстановить basic auth — генерировать новый hash.