chrisp/brackt
1
1
Fork 0
Code Issues 42 Pull requests Projects Releases Packages Wiki Activity Actions

Debug sync-and-simulate cron: show curl output, log auth failures #83

Merged
chrisp merged 1 commit from fix/sync-simulate-debug into main 2026-06-10 16:50:46 +00:00
Conversation 0 Commits 1 Files changed 2 +4 -1
chrisp commented 2026-06-10 16:45:20 +00:00
Owner
Copy link

Summary

  • Removes -s (silent) from the curl call in the workflow so CI logs show the HTTP response body on failure — previously all output was suppressed, making it impossible to tell whether the failure was a bad secret, missing env var, or network error
  • Adds logger.error in requireCronSecret so auth failures route to Sentry in production and are visible in dev logs

Test plan

  • Trigger a run (or wait for the next 2-hour schedule) and confirm the CI log now shows the response body
  • Verify CRON_SECRET is set in Forgejo repo secrets and matches the server env var
  • Confirm a successful run returns HTTP 200 with a JSON body

🤖 Generated with Claude Code

## Summary - Removes `-s` (silent) from the `curl` call in the workflow so CI logs show the HTTP response body on failure — previously all output was suppressed, making it impossible to tell whether the failure was a bad secret, missing env var, or network error - Adds `logger.error` in `requireCronSecret` so auth failures route to Sentry in production and are visible in dev logs ## Test plan - [ ] Trigger a run (or wait for the next 2-hour schedule) and confirm the CI log now shows the response body - [ ] Verify `CRON_SECRET` is set in Forgejo repo secrets and matches the server env var - [ ] Confirm a successful run returns HTTP 200 with a JSON body 🤖 Generated with [Claude Code](https://claude.com/claude-code)
chrisp added 1 commit 2026-06-10 16:45:20 +00:00
Debug sync-and-simulate cron job: surface curl output and log auth failures
All checks were successful
🚀 Deploy / 🧪 Test (pull_request) Successful in 3m3s
Details
🚀 Deploy / ʦ🔍 Typecheck & Lint (pull_request) Successful in 1m35s
Details
🚀 Deploy / 🐳 Build (pull_request) Has been skipped
Details
🚀 Deploy / 🚀 Deploy (pull_request) Has been skipped
Details
ff08155170 
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>
chrisp force-pushed fix/sync-simulate-debug from ff08155170 to 97a92aba7d 2026-06-10 16:50:33 +00:00 Compare
chrisp merged commit 97a92aba7d into main 2026-06-10 16:50:46 +00:00
chrisp deleted branch fix/sync-simulate-debug 2026-06-10 16:50:46 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
admin

Admin page tooling

  
defect

Something isn't working

  
draft

Related to the draft room

  
feature

New feature or request

  
infrastructure

Related to database/site infrastructure

  
new sport

Issue to add a new sport to the rotation

  
security

Things that can cause security issues

  
tech debt

Cleaning up things that will make us have a better life

  
ux

User interface or marketing

No labels
admin
defect
draft
feature
infrastructure
new sport
security
tech debt
ux
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: chrisp/brackt#83
No description provided.
Delete branch "fix/sync-simulate-debug"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?