getsentry / getsentry/sentry-javascript
bun: `Bun.serve` has no `ignoreStatusCodes`
- Vorherrschende Sprache
- TypeScript
- Sterne
- 8.7k
- Forks
- 1.8k
- Ø Merge
- 1 T. 17 Std.
- Gemergte PRs (30 T.)
- 515
Beschreibung
`denoServeIntegration` exposes `ignoreStatusCodes` and runs `processHttpServerTransactionEvent`; `httpServerSpansIntegration` does the same on Node. `bunServerIntegration` has no options at all, so 301/302/401/404 transactions are all recorded on Bun.
**Work item.** Add `ignoreStatusCodes` to `bunServerIntegration`, defaulting to `DEFAULT_IGNORE_STATUS_CODES`, and add the `processEvent` hook.
**Prior art ***(related history only)*. No issue for Bun. The Deno equivalents shipped as getsentry/sentry-javascript#23313 and getsentry/sentry-javascript#23692 (both merged), which aligned `denoHttpIntegration` options with `httpIntegration`. Bun was not part of either.
Beitragsleitfaden
Rechercherichtung
Start at bunServerIntegration and compare its options and transaction handling with denoServeIntegration and httpServerSpansIntegration. Trace DEFAULT_IGNORE_STATUS_CODES and the processEvent hook, then verify that Bun supports configuring ignored status codes and no longer records the specified status-code transactions by default.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- bun, typescript
- Bereich
- backend, observability-sre
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Aktiv
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 72/100