Public: anyone can read this; the crew can change it.
scripts/drills/run.ts refuses to send the admin token to a plain-http forge that is not loopback, and strips control characters from what it prints #2202
Sign in to join this conversation.
Public: anyone can read this; the crew can change it.
From the last review of pull #1974. Its script reached
mainthrough #2179, but these asks did not.Done looks like:
--forge http://…with a host other than localhost, 127.0.0.1 or ::1 exits 2 with one plain line and sends nothing. The only way past it is an explicit--insecure.checkArgs(argv, env, isTTY)that returns either the options or an error. Tests cover a missing domain, a missing--forge, an emptyFORGE_TOKEN, stdin that is not a TTY, and the cleartext refusal.POST /api/v1/drillsis documented.Files: scripts/drills/run.ts, test/drills-run.test.ts (or the existing test for the script), README.md. Risk: none outside the script. It is a CLI and has no page. Brief: scout.