Skip to content

fix: throttle transactions feed - #25

Open
NateIsern wants to merge 1 commit into
mainfrom
codex/propose-fix-for-unthrottled-transactions-feed
Open

fix: throttle transactions feed#25
NateIsern wants to merge 1 commit into
mainfrom
codex/propose-fix-for-unthrottled-transactions-feed

Conversation

@NateIsern

Copy link
Copy Markdown
Member

Motivation

  • The plural /api/transactions feed performs expensive daemon RPC fan-out (default getrawmempool + up to 20 getmempoolentry calls) but was only covered by the permissive global limiter, allowing unauthenticated clients to generate large backend load; this change bounds per-IP request cost.

Description

  • Apply the existing strictLimiter to the /api/transactions route in server/index.ts so the feed is protected the same way as other expensive RPC-backed endpoints.

Testing

  • Ran npm test which completed successfully (all test files passed).
  • Ran npm run typecheck which succeeded.
  • Attempted npm test -- --runInBand which failed because Vitest does not support the --runInBand flag, and git diff --check returned clean results.

Codex Task

@cursor

cursor Bot commented Aug 10, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant