fix(tradein/security): admin.py — generic detail + error-id вместо сырого текста исключений (#2234) #2261

Merged

lekss361 merged 2 commits from fix/tradein-admin-exception-leak into main

2026-07-03 06:27:23 +00:00

lekss361 commented

2026-07-03 06:22:06 +00:00

Owner

Summary

Единый хелпер _safe_http_error(status_code, public_detail, log_context): короткий error-id (uuid4().hex[:8]), полный traceback — только в logger.exception (→ GlitchTip), в HTTP — generic detail вида "fetch failed (error-id: abc12345)".
Заменены все 8 мест detail=f"...: {e}" (бывшие строки 551/557/580/586/629/669/672/679) — psycopg/httpx-исключения с DSN-фрагментами/внутренними хостами больше не утекают в ответы. Статус-коды сохранены 1:1 (502/500/404), контекст логирования (house_url/item_url/run_id/address) сохранён.
Broad-scan остального admin.py: других мест с exception-текстом в detail нет (проверено ревьюером).
По minor ревьюера: ветка IMVAddressNotFoundError (404, ожидаемое клиентское условие) логируется logger.warning без traceback — не создаёт exception-шум в GlitchTip.
Тест: exception с «секретным» DSN (postgres://user:SECRETPASS@...) → 502, в теле ответа нет секрета, есть error-id.

Как проверено

Полный uv run pytest (deselect только test_search_cache_hit): 3021 passed, 6 skipped; после minor-правки tests/test_scraper_admin_apis.py — 22 passed.
code-reviewer: ✅ APPROVE (все 8 вызовов в except-контексте верифицированы, raise ... from e, статус-коды сверены построчно).

Closes #2234

## Summary - Единый хелпер `_safe_http_error(status_code, public_detail, log_context)`: короткий error-id (`uuid4().hex[:8]`), полный traceback — только в `logger.exception` (→ GlitchTip), в HTTP — generic detail вида `"fetch failed (error-id: abc12345)"`. - Заменены все 8 мест `detail=f"...: {e}"` (бывшие строки 551/557/580/586/629/669/672/679) — psycopg/httpx-исключения с DSN-фрагментами/внутренними хостами больше не утекают в ответы. Статус-коды сохранены 1:1 (502/500/404), контекст логирования (house_url/item_url/run_id/address) сохранён. - Broad-scan остального admin.py: других мест с exception-текстом в detail нет (проверено ревьюером). - По minor ревьюера: ветка `IMVAddressNotFoundError` (404, ожидаемое клиентское условие) логируется `logger.warning` без traceback — не создаёт exception-шум в GlitchTip. - Тест: exception с «секретным» DSN (`postgres://user:SECRETPASS@...`) → 502, в теле ответа нет секрета, есть error-id. ## Как проверено - Полный `uv run pytest` (deselect только `test_search_cache_hit`): **3021 passed, 6 skipped**; после minor-правки `tests/test_scraper_admin_apis.py` — 22 passed. - code-reviewer: ✅ APPROVE (все 8 вызовов в except-контексте верифицированы, `raise ... from e`, статус-коды сверены построчно). Closes #2234

Rows
Columns