tradein/auth: устаревший текст ошибки 401 — «Caddy basic_auth required» после cutover #2573

New issue

Closed

opened 2026-07-30 21:56:08 +00:00 by lekss361 · 1 comment

lekss361 commented

2026-07-30 21:56:08 +00:00

Owner

Мелочь, замеченная при post-deploy проверке #2569.

Запрос к API Меры без сессии отдаёт:

GET https://gendsgn.ru/trade-in/api/v1/me
401 {"detail":"no authenticated user (Caddy basic_auth required)"}

После снятия basic_auth с /trade-in формулировка вводит в заблуждение: basic_auth там больше не требуется и не существует, нужна сессия приложения. Правильный текст — что-то вроде "valid session required" (как уже отдаёт team-API) или «войдите в систему».

Место: tradein-mvp/backend/app/core/rbac.py, ветка отсутствующего идентификатора. Проверить, не завязаны ли на эту строку тесты/фронт (грепнуть basic_auth required по репо), и поправить вместе с ними.

Влияния на работу нет — только на понятность для того, кто увидит ответ API.

Мелочь, замеченная при post-deploy проверке #2569. Запрос к API Меры без сессии отдаёт: ``` GET https://gendsgn.ru/trade-in/api/v1/me 401 {"detail":"no authenticated user (Caddy basic_auth required)"} ``` После снятия basic_auth с `/trade-in` формулировка вводит в заблуждение: basic_auth там больше не требуется и не существует, нужна сессия приложения. Правильный текст — что-то вроде `"valid session required"` (как уже отдаёт team-API) или «войдите в систему». Место: `tradein-mvp/backend/app/core/rbac.py`, ветка отсутствующего идентификатора. Проверить, не завязаны ли на эту строку тесты/фронт (грепнуть `basic_auth required` по репо), и поправить вместе с ними. Влияния на работу нет — только на понятность для того, кто увидит ответ API.

Light1YT referenced this issue from a commit

2026-08-05 08:42:08 +00:00

fix(tradein/auth): убрать устаревшее «Caddy basic_auth required» из 401 (#2573)

bot-backend referenced this issue

2026-08-05 08:42:24 +00:00

fix(tradein/auth): убрать устаревшее «Caddy basic_auth required» из 401 (#2573) #2649

bot-backend referenced this issue from a commit

2026-08-05 08:45:59 +00:00

fix(tradein/auth): убрать устаревшее «Caddy basic_auth required» из 401 (#2573) (#2649)

bot-backend commented

2026-08-05 08:51:32 +00:00

Collaborator

Сделано в PR #2649 (merged + live-verified).

Прод сейчас: GET https://gendsgn.ru/trade-in/api/v1/me → {"detail":"no authenticated user (valid session required)"}.

Формулировка не выдумана — приведена к той, что уже отдают team-API (current_team_actor) и dual-off ветка самого rbac_guard. Лид «no authenticated user» сохранён: на эту подстроку матчатся существующие тесты. Исправлены все 4 вхождения в tradein (core/rbac.py, api/v1/me.py, api/v1/trade_in.py ×2).

Проверено по грепу: та же строка есть в Site Finder (root backend/, 4 места) — там намеренно не тронута, basic_auth в Caddy для него жив и текст корректен. smoke-mera-perimeter.sh матчит только коды (не тело), доки/фронт литерал не используют — рассинхрона нет.

Сделано в PR #2649 (merged + live-verified). Прод сейчас: `GET https://gendsgn.ru/trade-in/api/v1/me` → `{"detail":"no authenticated user (valid session required)"}`. Формулировка не выдумана — приведена к той, что уже отдают team-API (`current_team_actor`) и dual-off ветка самого `rbac_guard`. Лид «no authenticated user» сохранён: на эту подстроку матчатся существующие тесты. Исправлены все 4 вхождения в tradein (`core/rbac.py`, `api/v1/me.py`, `api/v1/trade_in.py` ×2). Проверено по грепу: та же строка есть в Site Finder (root `backend/`, 4 места) — там **намеренно не тронута**, basic_auth в Caddy для него жив и текст корректен. `smoke-mera-perimeter.sh` матчит только коды (не тело), доки/фронт литерал не используют — рассинхрона нет.

bot-backend closed this issue

2026-08-05 08:51:34 +00:00