getsentry / getsentry/sentry-javascript

Improvements on `dataloader` integration

Offen
#13,724 1 Kommentar 0 Reaktionen 1 zugewiesene Person Beansprucht von @onurtemizkan Auf GitHub ansehen
Improvement Node.js
Vorherrschende Sprache
TypeScript
Sterne
8.7k
Forks
1.8k
Ø Merge
1 T. 17 Std.
Gemergte PRs (30 T.)
515

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.