getsentry / getsentry/sentry-javascript

Sentry can't instument express if it is bundled

オープン
#15,017 コメント 3 件 リアクション 0 件 担当者 0 名 GitHub で見る
Improvement Node.js
主要言語
TypeScript
スター
8.7k
フォーク
1.8k
平均マージ
1日 17時間
マージ済み PR(30日)
515

説明

### 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/node

### SDK Version

8.48.0

### Framework Version

Express 4.21.2

### Link to Sentry event

_No response_

### Reproduction Example/SDK Setup

_No response_

### Steps to Reproduce

Use sentry for node in an angular ssr appllication that is based on express.

### Expected Result

Sentry can instrument express

### Actual Result

Sentry comlains that express is not instrumented even through the instrument file was preloaded with `--import`.

My guess is that since angular bundles express into the server, sentry misses out on the import that it tries to wrap, leading to it not instrumenting express. I would be thankful for some ideas on how this could be improved.
Also if it is just adding a bunch of middlewares and such

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。