getsentry / getsentry/sentry-javascript

bun: `Bun.serve` has no `ignoreStatusCodes`

Abierto
#23,885 1 comentario 0 reacciones 0 asignados Ver en GitHub
Bun Improvement javascript
Lenguaje dominante
TypeScript
Estrellas
8.7k
Forks
1.8k
Merge medio
1 d 17 h
PR fusionados (30 d)
515

Descripción

`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.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

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.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
bun, typescript
Área
backend, observability-sre
Tipo de issue
Nueva funcionalidad
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Activo
Claridad
Bastante claro
Aptitud para principiantes
72/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.