getsentry / getsentry/sentry-javascript

Add `socketIoIntegration` to Node

Open
#13,310 2 comments 0 reactions 0 assignees View on GitHub
Improvement Node.js
Dominant language
TypeScript
Stars
8.7k
Forks
1.8k
Avg merge
1d 17h
Merged PRs (30d)
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
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.