fix(scrapers): named BrowserSession concurrency const, fix Semaphore(3) doc drift (#1347) #1913

Merged

bot-backend merged 1 commit from fix/browser-semaphore-doc-1347 into main

2026-06-26 07:44:30 +00:00

bot-backend commented

2026-06-26 07:32:02 +00:00

Collaborator

Резолвит doc/code drift из #1347. Issue в «Suggested fix» предлагает ДВА варианта: либо Semaphore(3), либо «обновить доки до 8 с обоснованием против наблюдаемой WAF-толерантности + сделать именованной константой». Выбран второй (issue-sanctioned) — т.к. issue сам констатирует, что 8 наблюдаемо WAF-толерантен, а срабатывание «вероятностное и пока не задокументировано как падающее» (severity medium, conf 0.85). Резать throughput в 2.7× без доказанных падений неоправданно.

stealth.py: константа _BROWSER_CONCURRENCY = 8 (комментарий: ramp 3→6→8, WAF-tolerant наш.дом.рф ServicePipe) → Semaphore(_BROWSER_CONCURRENCY); docstring обновлён.
domrf_kn.py: 2 устаревших комментария (Semaphore(3)) → (Semaphore(_BROWSER_CONCURRENCY)=8).

Runtime-значение не меняется (8). Код и доки больше не дрейфуют. 85 passed, ruff clean.

⚠️ Если по факту нужен throughput-cut до 3 ради WAF-safety (а не doc-fix) — это отдельное продуктовое решение с замером WAF-rate; этот PR фиксирует drift по второму варианту issue. Refs #1347

Резолвит doc/code drift из #1347. Issue в «Suggested fix» предлагает ДВА варианта: либо `Semaphore(3)`, либо **«обновить доки до 8 с обоснованием против наблюдаемой WAF-толерантности + сделать именованной константой»**. Выбран второй (issue-sanctioned) — т.к. issue сам констатирует, что 8 наблюдаемо WAF-толерантен, а срабатывание «вероятностное и пока не задокументировано как падающее» (severity medium, conf 0.85). Резать throughput в 2.7× без доказанных падений неоправданно. - `stealth.py`: константа `_BROWSER_CONCURRENCY = 8` (комментарий: ramp 3→6→8, WAF-tolerant наш.дом.рф ServicePipe) → `Semaphore(_BROWSER_CONCURRENCY)`; docstring обновлён. - `domrf_kn.py`: 2 устаревших комментария `(Semaphore(3))` → `(Semaphore(_BROWSER_CONCURRENCY)=8)`. Runtime-значение не меняется (8). Код и доки больше не дрейфуют. 85 passed, ruff clean. ⚠️ Если по факту нужен throughput-cut до 3 ради WAF-safety (а не doc-fix) — это отдельное продуктовое решение с замером WAF-rate; этот PR фиксирует drift по второму варианту issue. Refs #1347

Rows
Columns