Debug sync-and-simulate cron job: surface curl output and log auth failures
Remove -s (silent) from curl so CI logs show the HTTP response on failure. Add logger.error in requireCronSecret so 401s from bad/missing secrets appear in Sentry. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
parent
ffb1642ab6
commit
ff08155170
2 changed files with 4 additions and 1 deletions
|
|
@ -9,6 +9,6 @@ jobs:
|
|||
steps:
|
||||
- name: Sync standings and run simulations where changed
|
||||
run: |
|
||||
curl -sf -X POST https://brackt.com/admin/jobs/sync-and-simulate \
|
||||
curl -f -X POST https://brackt.com/admin/jobs/sync-and-simulate \
|
||||
-H "X-Cron-Secret: ${{ secrets.CRON_SECRET }}" \
|
||||
-H "Content-Type: application/json"
|
||||
|
|
|
|||
|
|
@ -1,7 +1,10 @@
|
|||
import { logger } from "~/lib/logger";
|
||||
|
||||
export function requireCronSecret(request: Request): void {
|
||||
const secret = process.env.CRON_SECRET;
|
||||
if (!secret) throw new Response("CRON_SECRET not configured", { status: 500 });
|
||||
if (request.headers.get("x-cron-secret") !== secret) {
|
||||
logger.error("[cron-auth] Invalid or missing X-Cron-Secret header");
|
||||
throw new Response("Unauthorized", { status: 401 });
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue