getsentry / getsentry/sentry-javascript

Improvements on `dataloader` integration

Đang mở
#13,724 1 bình luận 0 reaction 1 người được giao Được @onurtemizkan nhận Xem trên GitHub
Improvement Node.js
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ả

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

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.