Page MenuHomePhabricator

[token-server] Tighten rate limits
ClosedPublic

Authored by bytesofman on Sep 15 2024, 22:16.

Details

Reviewers
emack
Group Reviewers
Restricted Project
Commits
rABCd4fb4554f43e: [token-server] Tighten rate limits
Summary

Initially liked leaving the door open to users "gaming" this a bit, i.e. getting multiple manual claims. If they are really manual, it's not really a big deal -- implies someone wants to make another tx.

In practice though we are seeing scripts get most of the rewards. So, tighten it up.

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Branch
ts-ratelimit-again
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 30291
Build 60109: Build Difftoken-server-tests
Build 60108: arc lint + arc unit

Event Timeline

emack added a subscriber: emack.

Minor nit

apps/token-server/config.ts
56
This revision is now accepted and ready to land.Sep 16 2024, 01:10
This revision was automatically updated to reflect the committed changes.