getsentry / getsentry/sentry-javascript

Using both Bun and Express leads to "express is not instrumented" warning (Bun not publishing to http Diagnostic Channel)

Đang mở
#17,779 23 bình luận 5 reaction 1 người được giao Được @s1gr1d nhận Xem trên GitHub
Bug Bun External Dependency State: Blocked
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)
515

Mô tả

### Is there an existing issue for this?

- [x] I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
- [x] I have reviewed the documentation https://docs.sentry.io/
- [x] I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

### How do you use Sentry?

Sentry Saas (sentry.io)

### Which SDK are you using?

@sentry/bun

### SDK Version

10.15.0

### Framework Version

Express 5.1.0

### Link to Sentry event

_No response_

### Reproduction Example/SDK Setup

https://github.com/RudolfSchreier/sentry-express-bun-demo

### Steps to Reproduce

1. Get a working SENTRY_DSN for some project
2. start the demo using `env SENTRY_DSN=... bun run dev`

### Expected Result

The express server starts up with working Sentry integration and no warnings.

### Actual Result

The express server starts up, but with a Sentry warning about missing instrumentation:
> [Sentry] express is not instrumented. Please make sure to initialize Sentry in a separate file that you `--import` when running node, see: https://docs.sentry.io/platforms/javascript/guides/express/install/esm/.

### Additional Context

When using both Bun and Express, it is unclear which SDK to preferably use (@sentry/node or @sentry/bun), but both appear to show this issue.

It is unclear if the warning is correct or erroneous (and Express integration is actually working).

It is unclear to me from the description of https://github.com/getsentry/sentry-javascript/issues/13674 if this the expected result for Express v5.

Tip: React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding `+1` or `me too`, to help us triage it.

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

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

Đánh giá

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

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.