getsentry / getsentry/sentry-javascript

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

未关闭
#23,885 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
Bun Improvement javascript
主要语言
TypeScript
星标
8.7k
派生
1.8k
平均合并
1 天 17 小时
30 天内合并 PR
515

描述

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

贡献指南

打开贡献指南

调研方向

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.

由索引模型根据 Issue 内容生成。

评估

技术栈
bun, typescript
领域
backend, observability-sre
Issue 类型
功能
难度
3/5
预计耗时
1-2 天
活跃度
活跃
描述清晰度
基本清楚
新手友好度
72/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。