Skip to content

fix(api): disable response cache on miners, dashboard, and repos GET endpoints#214

Open
luciferlive112116 wants to merge 1 commit into
entrius:testfrom
luciferlive112116:fix/api-miners-dashboard-no-cache-201
Open

fix(api): disable response cache on miners, dashboard, and repos GET endpoints#214
luciferlive112116 wants to merge 1 commit into
entrius:testfrom
luciferlive112116:fix/api-miners-dashboard-no-cache-201

Conversation

@luciferlive112116

Copy link
Copy Markdown

Summary

  • Opt scoring-facing GET endpoints out of the global 60s response cache via @NoCache(), matching /pulls/.../files and /health.
  • Routes: GET /miners/:id/pulls, GET /miners/:id/issues, GET /dashboard/issues, GET /repos/:owner/:repo/maintainers.
  • Validators polling the mirror after webhooks no longer receive stale merge state, review counts, or labels for up to 60 seconds.

Fixes #201

Type of Change

  • Bug fix

Testing

  • Seven-line decorator-only change across three controllers; no logic changes.
  • CI: npm run build, npm run lint, npm run format:check, npm test in packages/das.

Checklist

  • I have read the Contributing Guide
  • Code builds without errors
  • No unnecessary dependencies added

…endpoints

Fixes entrius#201

Co-authored-by: Cursor <cursoragent@cursor.com>
@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] miners/dashboard/repos GET responses cached for 60s with no invalidation after webhooks

1 participant