Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Horizon rate limiting is not handled

Đã đóng
  • Có gì không ổn?
#348 2 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 2 ngày

@abdulrahmanj383-creator đang làm issue này rồi.

Từ ngày 16/10/2026.

  • #398 của @abdulrahmanj383-creator — đang mở
  • #467 của @sparkysolutions10-pixel — đang mở

Đánh giá

Issue này chưa được đánh giá.

Mô tả

area:ingest difficulty:intermediate help wanted points:150 Stellar Wave
Background

The AMM ingester calls Horizon directly:

const response = await fetch(
  `${getNetworkConfig(network).horizon.url}/liquidity_pools?${params.toString()}`
)
const data = await response.json() as any
if (!data._embedded?.records) return []

response.ok is never checked. A 429 from Horizon has no _embedded.records,
so it takes the same path as an empty result and the ingester reports that the
pair has no pools. Horizon publishes rate-limit headers and a Retry-After, and
none of them are read.

Under sustained rate limiting this ingester produces wrong data continuously
rather than backing off, and because of L163 the aggregate silently reweights
around it.

What to build
  • Check response.ok before parsing, everywhere Horizon is called.
  • Treat 429 distinctly: honour Retry-After when present, otherwise back off
    exponentially with jitter.
  • Surface remaining-quota headers as a gauge so approaching the limit is visible
    before it bites.
  • A 429 must not be reported as an empty venue — coordinate with L163.
Acceptance criteria
  • Non-2xx responses are detected before the body is parsed
  • 429 honours Retry-After when present and backs off otherwise
  • A rate-limited fetch is not reported as an empty result
  • Remaining quota is exported as a metric
  • Tests cover 429 with and without Retry-After

Drips Wave · Complexity: Intermediate · 150 points

Repo: Divinity-Labs/Lens. Node + TypeScript, Fastify, Prisma on Postgres,
Redis for queues, Prometheus via prom-client.

Run the tests: npm test. The tests are vitest, not jest. Running jest
against this repo reports "66 suites, 0 tests" and means nothing. A single file
is npx vitest run src/__tests__/yourfile.test.ts.

Load test (prices) is already failing on main and has been for some
time. It is not your PR that broke it. Every other check should be green.

This service prices real assets on Stellar mainnet. A number that is wrong
is worse than a number that is missing, because a caller cannot tell the
difference. Prefer failing loudly over returning a plausible value.

Never elide an address, a contract id or a key in code, a comment or a test
fixture. A previous batch wrote a shortened contract id in an issue text and
five contributors invented plausible-looking ids to fill the gap. Write the
whole thing or do not write it at all.

The house pattern for an outbound deadline already exists — see
src/alerts/delivery.ts, which passes signal: AbortSignal.timeout(10_000) to
fetch. Where an issue below asks for a timeout, copy that, do not invent a
second convention.

Required: Before submitting, join the contributor Telegram so your work can be tracked and counted toward the Stellar Wave: https://t.me/+fxHXq8f1SwlkZDBk

Ngôn ngữ chính
TypeScript
Star
4
Fork
71
Merge trung bình
2 ngày 20 giờ
Pull request đã merge (30 ngày)
127

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Divinity-Labs/Lens

Tất cả issue của Divinity-Labs/Lens

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.