GHSA-r5w7-f542-q2j4
Potential DoS when using ContextLines integration
En bref
- Publiée
- Mise à jour
- CVSS_V3
- CVSS:3.1/AV:N/AC:H/PR:N/UI:N/S:U/C:N/I:N/A:L
Description
### Impact The [ContextLines integration](https://docs.sentry.io/platforms/javascript/guides/node/configuration/integrations/contextlines/) uses readable streams to more efficiently use memory when reading files. The ContextLines integration is used to attach source context to outgoing events. The stream was not explicitly closed after use. This could lead to excessive amounts of file handles open on the system and potentially lead to a Denial of Service (DoS). The ContextLines integration is enabled by default in the Node SDK (`@sentry/node`) and SDKs that run in Node.js environments (`@sentry/astro`, `@sentry/aws-serverless`, `@sentry/bun`, `@sentry/google-cloud-serverless`, `@sentry/nestjs`, `@sentry/nextjs`, `@sentry/nuxt`, `@sentry/remix`, `@sentry/solidstart`, `@sentry/sveltekit`). ### Patches Users should upgrade to version `8.49.0` or higher. ### Workarounds To remediate this issue in affected versions without upgrading to version `8.49.0` and above you can disable the ContextLines integration. See the [docs](https://docs.sentry.io/platforms/javascript/guides/node/configuration/integrations/#removing-a-default-integration) for more details. ```js Sentry.init({ // ... integrations: function (integrations) { // integrations will be all default integrations return integrations.filter(function (integration) { return integration.name !== "ContextLines"; }); }, }); ``` If you disable the ContextLines integration, you will lose source context on your error events. ### References - Reported issue: https://github.com/getsentry/sentry-javascript/issues/14892 - PR Fix: https://github.com/getsentry/sentry-javascript/pull/14997
Paquets affectés
| Paquet | Versions |
|---|---|
| npm · @sentry/node | introduced 8.10.0, fixed 8.49.0 |
| npm · @sentry/astro | introduced 8.10.0, fixed 8.49.0 |
| npm · @sentry/aws-serverless | introduced 8.10.0, fixed 8.49.0 |
| npm · @sentry/bun | introduced 8.10.0, fixed 8.49.0 |
| npm · @sentry/google-cloud-serverless | introduced 8.10.0, fixed 8.49.0 |
| npm · @sentry/nestjs | introduced 8.10.0, fixed 8.49.0 |
| npm · @sentry/nextjs | introduced 8.10.0, fixed 8.49.0 |
| npm · @sentry/nuxt | introduced 8.10.0, fixed 8.49.0 |
| npm · @sentry/remix | introduced 8.10.0, fixed 8.49.0 |
| npm · @sentry/solidstart | introduced 8.10.0, fixed 8.49.0 |
| npm · @sentry/sveltekit | introduced 8.10.0, fixed 8.49.0 |
Références
- https://github.com/getsentry/sentry-javascript/security/advisories/GHSA-r5w7-f542-q2j4 WEB
- https://github.com/getsentry/sentry-javascript/issues/14892 WEB
- https://github.com/getsentry/sentry-javascript/pull/14997 WEB
- https://github.com/getsentry/sentry-javascript PACKAGE
Sources
| Source | Licence |
|---|---|
| OSV.dev | CC-BY 4.0 — GitHub Advisory Database |