getsentry / getsentry/sentry-javascript

Add `socketIoIntegration` to Node

Ouverte
#13,310 2 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Improvement Node.js
Langage dominant
TypeScript
Étoiles
8.7k
Forks
1.8k
Merge moyen
1 j 17 h
PR mergées (30 j)
515

Description

### Description

We want to instrument `socket.io`. There is an existing OTEL instrumentation for this:

https://www.npmjs.com/package/@opentelemetry/instrumentation-socket.io

```[tasklist]
### Tasks
- [ ] Add new `socketIoIntegration` in `@sentry/node`
- [ ] This should set up the OTEL instrumentation
- [ ] Make sure to add an `origin` to all spans the integration emits
- [ ] Add node-integration-tests for the package
- [ ] The integration should be added by default
```

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.