getsentry / getsentry/sentry-javascript
Improvements on `dataloader` integration
- Langage dominant
- TypeScript
- Étoiles
- 8.7k
- Forks
- 1.8k
- Merge moyen
- 1 j 17 h
- PR mergées (30 j)
- 515
Description
### 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
Guide de contribution
Ouvrir le guide de contribution
Évaluation
Cette issue n'a pas encore été évaluée.