bot-backend
cb5e2d4f70
All checks were successful
CI Trade-In / changes (pull_request) Successful in 7s
CI / changes (pull_request) Successful in 9s
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 2m16s
CI / backend-tests (pull_request) Successful in 17m10s
scrape_catalog_object гасил WafBlockedError и возвращал его как обычную неудачу, поэтому батч-цикл шёл дальше и слал ЖИВОЙ запрос на каждый оставшийся obj_id в уже забаненную сессию. Цена. Замер прода 20.08: в очереди 13200 объектов из 13801, а DOM.РФ отдаёт страницу «Доступ заблокирован [403]» с капчей (#2443). Один прогон «Загрузить все» выдал бы 13200 живых запросов в забаненную сессию — ровно то, что углубляет бан. Анти-бан-комментарий к BrowserSession в этом же файле предупреждает про тот же path family, из-за которого бан и случился. Правка из двух частей: per-object функция пробрасывает WAF (это не «объект не дошёл», а закрытая дверь), батч-цикл считает ПОДРЯД идущие блоки и прерывается на третьем. Порог не единица намеренно: одиночный блок бывает переходным, три подряд — стена. Прерванный батч помечается в stats (aborted_on_waf), а stats уходит в результат задачи через **stats — видно в логе прогона. Про двусторонность честно: части проверяются по-разному. • Проброс WAF — ДВУСТОРОННЕ. Тест подменяет fetch_catalog_object_html (настоящий источник блока) и гоняет НАСТОЯЩУЮ функцию: на origin/main «DID NOT RAISE», потому что там исключение гасится. • Предохранитель — логики этой на origin/main нет, поэтому тесты подменяют scrape_catalog_object и меряют новый цикл в изоляции (сколько объектов он успел тронуть). На двустороннюю проверку они не претендуют, и это написано в шапке файла. Первая версия теста этого не различала: подменяла scrape_catalog_object и на main падала с пробросом исключения — то есть по «нет такого кода», а не по неверному поведению. Перестроено. Контроли: одиночный блок не рвёт прогон; успех между блоками сбрасывает счётчик; обычный сбой сети по-прежнему гасится в False (иначе один битый объект рвал бы батч); здоровый батч проходит все объекты. Прогоны: tests/services/scrapers + tests/workers — 525 passed rc=0. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| analysis_runs | ||
| cadastre | ||
| chat | ||
| exporters | ||
| forecasting | ||
| generative | ||
| llm | ||
| scrapers | ||
| site_finder | ||
| __init__.py | ||
| test_2464_docs_match_code.py | ||
| test_2464a_job_settings_savepoint.py | ||
| test_2464h_timing_overlap_formula.py | ||
| test_analytics_queries_domrf_dedup.py | ||
| test_cadastre_bulk.py | ||
| test_dadata_client.py | ||
| test_ekburg_permits.py | ||
| test_forecast_request_cache.py | ||
| test_functional_zone_lookup.py | ||
| test_granddoc_lookup.py | ||
| test_ird_analyze.py | ||
| test_ird_overlay_lookup.py | ||
| test_job_settings.py | ||
| test_krt_lookup.py | ||
| test_mapping_review.py | ||
| test_newbuilding_crossload.py | ||
| test_nspd_denorm.py | ||
| test_obj_class_backfill.py | ||
| test_objective_backfill.py | ||
| test_own_developer_ids_config.py | ||
| test_planning_lookup.py | ||
| test_ppt_tep_lookup.py | ||
| test_quarter_dump_lookup.py | ||
| test_recommend_mix_velocity.py | ||
| test_reservation_lookup.py | ||
| test_utility_characteristics.py | ||
| test_weather_cache.py | ||
| test_zone_regulation_backfill.py | ||
| test_zone_regulation_extract.py | ||
| test_zone_regulation_memo.py | ||
| test_zone_regulation_write_session.py | ||