Correct UniswapX filler orders API rate limit to 4 RPS - #1158
Open
claude[bot] wants to merge 1 commit into
Open
Conversation
The filler FAQ still told fillers to rate limit at 6 RPS, and the filler
overview table repeated it. The deployed limit on the UniswapX orders
endpoint was lowered on 2026-09-02, and the developer portal has published
4 RPS since 2026-09-03, so both pages here were stale.
Updates the four occurrences in content/liquidity/uniswapx/filling/faq.mdx
(two in the FAQ frontmatter, two in the page body) and the one in the filler
role table in content/liquidity/uniswapx/filling/overview.mdx, and mirrors
the developer portal wording on the "How do I get orders?" answer: cached
responses, 429 above the limit, and at most the last 50 orders per response.
Also ports the developer portal's "Do I need an API key to poll for orders?"
entry, which this mirror was missing. The orders API is a separate service
from the Trading API and needs no API key, and the limit is keyed on the
client IP, so a Trading API key's 6 RPS per-key limit (documented under
content/trading/swapping-api/) does not apply to order polling and raising it
does not increase polling throughput. Those trading pages are unchanged: their
per-key figure is correct for the Trading API and matches the developer portal.
Left the archived copy under archive/ alone, per CONTRIBUTING.md ("the
archive/ directory holds the previous version of this repository for
reference only").
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013WuaPUrQTFBfsT9rk7FXpa
|
Thanks for contributing to Uniswap Docs! 馃 Quick heads up on how this repo works: the docs you see on developers.uniswap.org are built from a separate source repo. When we accept a change here, we port it over there, so your fix might show up on the live site before it lands in this repo. Updates to this repo come in batches on a regular cadence. When yours is included, you'll be credited as a co-author on the commit, so the contribution counts on your GitHub profile and in Uniswap/docs. We review every PR but can't merge everything. Either way, someone from the docs team will follow up here once we've taken a look. One legal bit: by submitting a PR you agree your contribution is licensed under the repo's MIT license. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by Angela Ocando 路 Slack thread
Summary
The UniswapX filler pages still publish a 6 RPS polling limit for the orders API. The deployed limit was lowered on 2026-09-02 and the published figure is now 4 RPS, so a filler following these pages polls above the limit and gets
429s.429, and each response returns at most the last 50 orders, ordered by most recent.Before and after, "How do I get orders?":
Two things deliberately left alone:
content/trading/swapping-api/common-errors.mdxandfaqs.mdx, which document a default of 6 requests per second per API key. That is the Trading API's per-key limit, a different service from the orders endpoint, and it matches what the developer portal publishes today. The new FAQ entry above exists to keep the two from being read as the same limit.archive/docs/contracts/uniswapx/fillers/filleroverview.md, which says "6rps".CONTRIBUTING.mdstates thatarchive/holds the previous version of the repository for reference only and that PRs against it are not accepted, so the frozen copy stays as it is.Type of change
How has this been verified?
{"errorCode": "TOO_MANY_REQUESTS"}above it.faq.mdxparses, and no em-dashes were introduced.Applicable screenshots
N/A, prose only.
Anything else reviewers should know?
The 6 RPS figure was correct for the orders endpoint until 2026-09-02, so anything else sourced from before that date is worth a second look. The stale figure also caused real confusion: a filler read a per-API-key limit elsewhere in these docs and concluded that using a different key would raise its polling throughput, which the new FAQ entry now rules out explicitly.
馃 Generated with Claude Code
https://claude.ai/code/session_013WuaPUrQTFBfsT9rk7FXpa
Generated by Claude Code