Skip to content

Audit 2026-10-05: CDP failure hint by cause; atomic output writes - #11

Merged
jehrr merged 2 commits into
mainfrom
audit-2026-10-05
Oct 5, 2026
Merged

jehrr merged 2 commits into
mainfrom
audit-2026-10-05

Conversation

@jehrr

@jehrr jehrr commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator

Two defects confirmed from a third-party audit, each with a planted-fault control (both went red on the named check; suite green after restore).

  • CDP hint: Playwright appended the profile_locked/pid advice to every connect error. Now chosen by cause (401, DNS, 500/locked, timeout, unknown) via page_flow.cdp_failure_hint.
  • Atomic writes: write_json/write_csv/write_run_meta truncated the previous good file before writing. Now temp file + fsync + os.replace, mode preserved (not left 0600).

Not changed, and why: the canary cron stays off (profile credentials last ~a day; see canary.yml header); --concurrency Playwright-only is documented and warned; the audit's ENOTFOUND was its own sandbox (cb.2captcha.com resolves here).

🤖 Generated with Claude Code

petr and others added 2 commits October 5, 2026 14:58
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@jehrr
jehrr marked this pull request as ready for review October 5, 2026 17:25
@jehrr
jehrr merged commit 5fb370a into main Oct 5, 2026
8 checks passed
@jehrr
jehrr deleted the audit-2026-10-05 branch October 5, 2026 17:25
@jehrr jehrr mentioned this pull request Oct 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant