bot-backend
163bacb7be
All checks were successful
CI Trade-In / changes (pull_request) Successful in 7s
CI / changes (pull_request) Successful in 8s
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 / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m11s
CI / backend-tests (pull_request) Successful in 15m33s
`verify_admin_token` / `AdminTokenAuth` в `backend/app/core/deps.py` не имели НИ ОДНОГО вызывающего. Проверено разбором AST по всему репозиторию (Name / Attribute / ImportFrom / строковые константы — формы, которые текстовый grep не ловит): оба имени встречаются ровно в одном месте — в собственном объявлении, строка 23 самого deps.py. Больше нигде. Поведение сняли в PR #437 (заголовок X-Admin-Token убран со всех admin_*.py), UI ввода токена — в PR #442. Поле держали «для быстрого rollback» полтора месяца. Rollback это всё равно не ускоряло: dep нигде не проставлен, вернуть его = править все admin_*.py, ровно как и было бы без сохранённого поля. Что реально закрывает `/api/v1/admin/*` — два живых слоя: 1. middleware `rbac_guard` (app/main.py): `_ADMIN_API_RE` → 403 `admin only` при role != admin. Покрыт tests/test_rbac.py — 24 теста, все зелёные. 2. Caddy basic_auth на весь gendsgn.ru (PR #426). Удалено: • backend/app/core/deps.py целиком (в файле не было ничего, кроме этого хелпера; импортеров модуля тоже нет); • `scrape_admin_token` из Settings — его единственным читателем был удалённый хелпер; • SCRAPE_ADMIN_TOKEN из .env.example. Прод-.env править не обязательно: `model_config` = `extra="ignore"`, лишняя переменная безвредна; • остатки в тестах: 10 декораторов `@patch(...scrape_admin_token)` и 25 `headers=ADMIN_HEADERS` — заголовок отправлялся во все admin-запросы и никем не читался. Попутно исправлены ДВА докстринга, обещавшие покрытие, которого нет: test_admin_weight_profiles.py («401 при отсутствии X-Admin-Token») и test_admin_ekburg_permits.py («отсутствие X-Admin-Token → 401/503»). Соответствующего теста нет ни в одном из файлов — ни сейчас, ни до этого коммита. Теперь докстринги указывают на tests/test_rbac.py, где проверка действительно есть. Верификация: полный сьют backend без tests/smoke — 4610 passed, 48 skipped, 0 failed (12м38с). Целевые файлы отдельно: 51 passed (24 rbac + 27 admin). Closes #2775 |
||
|---|---|---|
| .. | ||
| api | ||
| fixtures | ||
| integration | ||
| scrapers | ||
| scripts | ||
| services | ||
| smoke | ||
| sql | ||
| workers | ||
| __init__.py | ||
| conftest.py | ||
| skip_allowlist.txt | ||
| test_admin_weight_profiles.py | ||
| test_app_logging.py | ||
| test_audit_middleware.py | ||
| test_auth_db.py | ||
| test_auth_session_guard.py | ||
| test_auth_session_service.py | ||
| test_concepts_stub.py | ||
| test_connection_capacity_loaders.py | ||
| test_eesk_reserve_loader.py | ||
| test_eias_heat_loader.py | ||
| test_ekb_geoportal_client.py | ||
| test_ekgas_outlet_loader.py | ||
| test_gas_grs_loader.py | ||
| test_gate_verdict.py | ||
| test_gisogd66_loader.py | ||
| test_health.py | ||
| test_layout_signature.py | ||
| test_layout_tz_html.py | ||
| test_layout_tz_pdf.py | ||
| test_network_obremenenie.py | ||
| test_network_zone_humanize.py | ||
| test_noise_loader.py | ||
| test_nspd_client.py | ||
| test_nspd_sync.py | ||
| test_ors_client.py | ||
| test_osm_tp_geocoder.py | ||
| test_osrm_client_local.py | ||
| test_ping.py | ||
| test_poi_score.py | ||
| test_quarter_dump_lookup.py | ||
| test_rbac.py | ||
| test_riasurt_sverdl_client.py | ||
| test_saturation.py | ||
| test_sentry_init.py | ||
| test_utility_infrastructure_loader.py | ||
| test_velocity.py | ||
| test_weight_profiles.py | ||