getsentry / getsentry/sentry-javascript

Improvements on `dataloader` integration

オープン
#13,724 コメント 1 件 リアクション 0 件 担当者 1 名 @onurtemizkan が担当を希望しています GitHub で見る
Improvement Node.js
主要言語
TypeScript
スター
8.7k
フォーク
1.8k
平均マージ
1日 17時間
マージ済み PR(30日)
515

説明

### Problem Statement

Following up: https://github.com/getsentry/sentry-javascript/pull/13664

https://github.com/getsentry/sentry-javascript/pull/13664 introduced basic support for `dataloader` integration on Sentry Node SDK.

In its current state, the feature set is limited, as there's no hook we can get request / lookup information from the underlying instrumentation [@opentelemetry/instrumentation-dataloader](https://www.npmjs.com/package/@opentelemetry/instrumentation-dataloader).

More info on this: https://github.com/getsentry/sentry-javascript/pull/13664#issue-2518420565

### Solution Brainstorm

We can open PRs on https://github.com/open-telemetry/opentelemetry-js-contrib/

- [ ] Adding a `requestHook` or `lookupHook` to the OTEL instrumentation
- [x] #13729

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

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

評価

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

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

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