diff --git a/tradein-mvp/browser/Dockerfile b/tradein-mvp/browser/Dockerfile index e8239d08..d98b761c 100644 --- a/tradein-mvp/browser/Dockerfile +++ b/tradein-mvp/browser/Dockerfile @@ -23,26 +23,33 @@ WORKDIR /app # camoufox[geoip] тянет playwright как зависимость RUN pip install --no-cache-dir "camoufox[geoip]" aiohttp -# ── #915: null-guard для pageError.location в playwright 1.60.0 ───────────────── +# ── #915/#918: null-guard + ВАЛИДНЫЕ дефолты для pageError.location (playwright 1.60.0) ─ # Firefox/juggler-бэкенд иногда отдаёт uncaught JS-ошибку страницы (реклама / -# cross-origin скрипты на avito) БЕЗ поля location. playwright-диспетчер при -# доставке pageError-события читает `pageError.location.url` (coreBundle.js, -# 2 места: BrowserContextDispatcher + trace event) без null-guard → -# `TypeError: Cannot read properties of undefined (reading 'url')` ВНУТРИ -# event-хэндлера node-драйвера → рвёт protocol-соединение → "Connection closed -# while reading from the driver" / TargetClosedError → server.py отдаёт 500 и -# делает полный relaunch браузера (дорого, тормозит sweep). Недетерминированно: -# зависит от того, кинет ли конкретная загрузка locationless-ошибку. -# Апгрейд playwright невозможен — camoufox 0.4.11 pinned под конкретный -# Firefox+juggler. Чиним optional-chaining'ом прямо в pinned-бандле: -# `pageError.location.X` → `pageError.location?.X` (no-op когда location есть; -# отдаёт {url:undefined,...} когда нет → событие доставляется без краша). -# Сборка падает (exit 1) если паттерн не найден — сигнал что bump playwright -# сломал патч. +# cross-origin скрипты / anti-bot challenge на avito) БЕЗ поля location. +# playwright при доставке pageError-события собирает +# { url: pageError.location.url, line: ...lineNumber, column: ...columnNumber } +# (coreBundle.js, 2 места: BrowserContextDispatcher + trace event). +# +# Двухслойный баг (оба рвут protocol-соединение → "Connection closed while reading +# from the driver" / TargetClosedError → server.py 500 + relaunch браузера): +# 1. без guard → `pageError.location.url` бросает +# `TypeError: Cannot read properties of undefined (reading 'url')`; +# 2. ОДНОГО optional-chaining (`?.url`) НЕДОСТАТОЧНО (это была дыра #918): +# url станет undefined, а protocol-валидатор `tString` требует string → +# `ValidationError: location.url: expected string, got undefined`. +# Поэтому даём валидные дефолты: url → '', line/column → 0 — проходят валидатор, +# событие доставляется без краша. Профит: browser возвращает HTML challenge- +# страницы вместо краша → pipeline видит firewall и штатно ротирует IP, а не +# уходит в crash-storm (подтверждено прод-дебагом локального стека 2026-05-31). +# +# Апгрейд playwright невозможен — camoufox 0.4.11 pinned под Firefox+juggler. +# Сборка падает (exit 1), если паттерн не найден — сигнал что bump playwright сломал патч. RUN PW_BUNDLE="$(python -c 'import os,playwright;print(os.path.join(os.path.dirname(playwright.__file__),"driver","package","lib","coreBundle.js"))')" \ - && sed -i -E 's/pageError\.location\.(url|lineNumber|columnNumber)/pageError.location?.\1/g' "$PW_BUNDLE" \ - && N="$(grep -cF 'pageError.location?.url' "$PW_BUNDLE")" \ - && echo "tradein-browser: patched pageError.location?.url sites: $N" \ + && sed -i -E "s/pageError\.location\.url,/pageError.location?.url ?? '',/g" "$PW_BUNDLE" \ + && sed -i -E 's/pageError\.location\.lineNumber,/pageError.location?.lineNumber ?? 0,/g' "$PW_BUNDLE" \ + && sed -i -E 's/pageError\.location\.columnNumber/pageError.location?.columnNumber ?? 0/g' "$PW_BUNDLE" \ + && N="$(grep -cF "pageError.location?.url ?? ''" "$PW_BUNDLE")" \ + && echo "tradein-browser: patched pageError.location?.url ?? '' sites: $N" \ && [ "$N" -ge 2 ] COPY server.py ./server.py