Skip to content

fix: throttle transactions feed - #25

Merged
NateIsern merged 2 commits into
mainfrom
codex/propose-fix-for-unthrottled-transactions-feed
Sep 10, 2026
Merged

NateIsern merged 2 commits 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.

@NateIsern
NateIsern merged commit e15a848 into main Sep 10, 2026
1 check passed
@NateIsern
NateIsern deleted the codex/propose-fix-for-unthrottled-transactions-feed branch September 10, 2026 15:10
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