getsentry / getsentry/sentry-javascript

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

Đang mở
#23,885 1 bình luận 0 reaction 0 người được giao Xem trên GitHub
Bun Improvement javascript
Ngôn ngữ chính
TypeScript
Star
8.7k
Fork
1.8k
Merge trung bình
1 ngày 17 giờ
Pull request đã merge (30 ngày)
523

Mô tả

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

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

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.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
bun, typescript
Lĩnh vực
backend, observability-sre
Loại issue
Tính năng
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Sôi nổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
72/100

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.