OperatorsSafety
Security review
The checklist and where it stands: all eight P0 items pass, and the log scan plus the two judge-room items are next.
Checklist SEC-01 to SEC-24, the same 24 rows as docs/SECURITY-REVIEW.md. Each item is PASS, PARTIAL (the check passes and a named residual is still open), FAIL (with an issue link), ACCEPTED (a risk kept on purpose, with a justification) or N/A. A P0 FAIL blocks submission. The reviewer of an item is never its author. Status OPEN means nobody has signed the item off yet. Rows marked agent-only were reviewed by an agent from the code and tests, with no human reviewer yet.
| ID | Priority | Area | Status | Automated check |
|---|---|---|---|---|
| SEC-01 | P0 | Access control on auctioneer actions | PASS | packages/e2e/test/security/access.test.ts |
| SEC-02 | P0 | Bidder identity and token binding | PASS | access.test.ts |
| SEC-03 | P0 | authenticity | PASS | webhook.test.ts |
| SEC-04 | P0 | Secrets in repo, logs and bundle | PASS | gitleaks workflow, bundle scan |
| SEC-05 | P0 | -only guard and boot validation | PASS | sandbox.test.ts |
| SEC-06 | P0 | Money integrity (integer cents, ) | PASS | engine and payments tests |
| SEC-07 | P0 | Injection and rendering of untrusted text | PASS | fuzz.test.ts |
| SEC-08 | P1 | Abusive input caps | PASS | abuse.test.ts |
| SEC-09 | P1 | CORS and origin allowlist | PARTIAL | T-QA-075 |
| SEC-10 | P1 | Transport and security headers, CSP | PARTIAL | headers.test.ts, web network.spec.ts |
| SEC-11 | P1 | Audio and personal data | ACCEPTED | manual review |
| SEC-12 | P1 | Dependency risk | PASS | audit.yml, audit-waivers.json |
| SEC-13 | P1 | SSRF | PASS | webhook.test.ts (certificate URL host allowlist) |
| SEC-14 | P1 | Denial of service on the parse path | PARTIAL | dos.test.ts, proxy hops probe (2 hops on Render, checked once on 9 Nov) |
| SEC-15 | P2 | Logging hygiene | PARTIAL | log scan after scenario S01 (run 42566851766) |
| SEC-16 | P2 | Admin and ops pages | PASS | manual review |
| SEC-17 | P2 | Clickjacking and PayPal return URL redirect | PASS | manual review |
| SEC-18 | P0 | Server time is authoritative | PASS | T-QA-079 |
| SEC-19 | P1 | Key-less judge rooms | OPEN | http-judge-rooms.test.ts (agent-only) |
| SEC-20 | P1 | Shareable receipt link | OPEN | http-receipt.test.ts, token.test.ts (agent-only) |
| SEC-21 | P1 | Judge key and lockout (x-judge-key) | PARTIAL | http-judge-key.test.ts, config-judge-key.test.ts (agent-only) |
| SEC-22 | P1 | Reviewer token and ledger route | PARTIAL | http-judge-key.test.ts, token.test.ts (agent-only) |
| SEC-23 | P1 | Simulated holds in judge rooms | PARTIAL | http-judge-sim-hold.test.ts, sim-holds.test.ts (agent-only) |
| SEC-24 | P2 | Docs host and custom-domain headers | PARTIAL | docsite/public/_headers, docsite workflow, docs/evidence/security/live-checks-20261109.md (agent-only) |
- docs/SECURITY-REVIEW.mdthe checklist and how to record a result
Edit on GitHub
Last updated on