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

A webhook that fails three times disappears

Đã đóng
  • Có gì không ổn?
#342 4 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

@Herikcool đang làm issue này rồi.

Từ ngày 16/10/2026.

  • #388 của @Herikcool — đang mở
  • #467 của @sparkysolutions10-pixel — đang mở

Đánh giá

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

Mô tả

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

When deliverJsonWithRetries exhausts its attempts it does this:

console.warn(`[webhook] delivery failed after ${attempt} attempts to ${url}: HTTP ${res.status}`)

…and returns. The alert is gone. The subscriber never learns that an alert fired,
and the operator learns only if someone is reading warn-level logs at the moment
it happens. For an alerting product this is the worst possible failure, because
silence is indistinguishable from "nothing happened".

What to build
  • Persist failed deliveries — endpoint, payload, attempt count, last status, first
    and last attempt times.
  • A counter metric of permanent delivery failures, labelled by a bounded category
    (not by URL — see L171).
  • A way to see recent failures, either on an admin route or via /status.
  • Do not store a delivery secret alongside the payload.
  • Decide and document whether failed deliveries can be replayed; building replay
    is optional, but say which it is.
Acceptance criteria
  • An exhausted delivery is recorded durably, not only logged
  • The record contains enough to diagnose without the original log line
  • A metric counts permanent failures with bounded label values
  • Secrets are not persisted
  • Recent failures are retrievable through an endpoint

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.