Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Webhook retries have no jitter, so failures synchronise

オープン 初心者向け
  • 何か問題がありますか？
#341 コメント 2 件 リアクション 0 件 担当者 1 名 GitHub で見る

メンテナーはふだん 2 日以内に返信

@Richard-tobi がすでに取り組んでいます。

2026年10月16日 から。

  • #428 @Richard-tobi による — マージされずにクローズ
  • #467 @sparkysolutions10-pixel による — マージされずにクローズ
  • #611 @Richard-tobi による — オープン

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
80/100
issue の種類
機能追加
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
nodejs, typescript
領域
backend

調査の方向性

src/alerts/delivery.ts から始めてください。ここでは retryDelayMs が決定的な 1 秒/2 秒/4 秒のバックオフを計算し、リトライループがそれを呼び出しています。注入可能な乱数源の背後に完全なジッターを追加してください。既存の試行回数の上限と、4xx レスポンスはリトライしないというルールは維持します。固定の乱数源を使い、npx vitest run src/tests/<file>.test.ts で単一の vitest ファイルを実行し、同時に発生した 2 つの失敗が異なるタイミングでリトライされることを示してください。遅延にジッターが入り、テストが決定的に成功し、4xx と試行回数上限の動作が変わっていなければ完了です。

索引モデルが issue の本文から書いたものです。

説明

area:reliability difficulty:easy help wanted points:100 Stellar Wave
Background

src/alerts/delivery.ts backs off deterministically:

function retryDelayMs(attempt: number): number {
  return Math.pow(2, attempt - 1) * 1000
}

Every delivery that fails at the same moment retries at exactly the same moment
— 1s, then 2s, then 4s. A price movement is precisely the event that triggers
many alerts at once, so when a subscriber's endpoint has a wobble, Lens responds
by hitting it with a synchronised burst, three times, which is a good way to
turn a wobble into an outage.

What to build
  • Add jitter. Full jitter — a random value between zero and the computed delay —
    is the standard choice and is one line.
  • Keep the existing retry count and the existing rule that 4xx is not retried,
    which is correct.
  • Make the test deterministic by injecting the randomness rather than asserting
    on a range.
Acceptance criteria
  • Retry delays are jittered
  • Two deliveries failing simultaneously retry at different times, asserted by a test
  • 4xx responses are still not retried
  • The attempt cap is unchanged
  • The randomness source is injectable so the test is not flaky

Drips Wave · Complexity: Easy · 100 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

主要言語
TypeScript
スター
4
フォーク
71
平均マージ
2日 20時間
マージ済み PR（30日）
127

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

Divinity-Labs/Lens のほかの issue

Divinity-Labs/Lens の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。