JMR-devandClaude Opus 4.8
047391d01c
Add an authenticated admin API to the ingest Worker so the single maintainer
can review the pending queue and pull a report before Friday's publish run.
Endpoints (on the existing handler):
GET /v1/admin/reports list pending report ids
POST /v1/admin/reports/{id}/remove mark a report removed
DELETE /v1/admin/reports/{id} remove alias
Remove calls lifecycle.MarkRemoved (#10), transitioning pending -> removed so
#13's ListPending excludes it from the next publish. Codes: 200 list/remove,
404 unknown id, 401 missing/bad/unset-secret token, 405 wrong method.
Auth: shared-secret Bearer token compared with crypto/subtle.ConstantTimeCompare,
fail-closed when the secret is unset. Injected via handler.New's new AdminBackend
arg: the dev server and tests wire a memory-backed lifecycle.Manager + ADMIN_TOKEN
env; the Worker reads ADMIN_TOKEN from Secrets Store and builds an R2-backed
Manager per request. Choice documented in docs/decisions/admin-auth.md.
Tests: Go httptest unit tests (list, remove+exclusion, 404, 401 incl. fail-closed,
405) and a Bruno api-tests flow (seed, authed list/remove, exclusion, no/bad
token 401). wrangler.jsonc gains only the ADMIN_TOKEN secret binding; worker
triggers untouched (owned by #13).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
33 lines
842 B
YAML
33 lines
842 B
YAML
info:
|
|
name: "admin: seed a pending report to review"
|
|
# Seeds one pending report so the admin list/remove flow below is self-contained
|
|
# (it does not depend on the earlier ingest contract tests having run first).
|
|
type: http
|
|
seq: 6
|
|
|
|
http:
|
|
method: POST
|
|
url: "{{baseUrl}}/v1/reports"
|
|
headers:
|
|
- name: Content-Type
|
|
value: application/json
|
|
body:
|
|
type: json
|
|
data: |-
|
|
{
|
|
"appVersion": "1.4.2 (142)",
|
|
"platform": "android",
|
|
"osVersion": "Android 14",
|
|
"device": "Pixel 7",
|
|
"clientTimestamp": "2026-07-02T12:34:56Z",
|
|
"report": "seed report for the admin review/removal flow (#11)"
|
|
}
|
|
|
|
runtime:
|
|
scripts:
|
|
- type: tests
|
|
code: |-
|
|
test("seed report is accepted (202)", function () {
|
|
expect(res.getStatus()).to.equal(202);
|
|
});
|