getsentry / getsentry/sentry-javascript

Add `socketIoIntegration` to Node

Abierto
#13,310 2 comentarios 0 reacciones 0 asignados Ver en GitHub
Improvement Node.js
Lenguaje dominante
TypeScript
Estrellas
8.7k
Forks
1.8k
Merge medio
1 d 17 h
PR fusionados (30 d)
515

Descripción

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

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.